release/index.json for v6.8.1 has invalid modules value

[phillipj]

As reported in nodejs/nodejs.org#969, the v6.8.1 release has modules: null in https://nodejs.org/download/release/index.json.

I assume the modules value originates from fetchModVersion() in dist-indexer.js. I've manually tested the regexes in that method against node_version.h and node.h, where the former seems to match perfectly fine, meaning it should not have resolved the modules value to null.

Any thoughts on how this might have happened? Could we execute dist-indexer.js manually to see if it was a temporary glitch, and the correct modules value gets resolved now?

[phillipj]

Just noticed v6.8.1 also has zlib: null

[targos]

It's also missing the npm property

[targos]

And uv is an empty string

[phillipj]

/cc @nodejs/release

[MylesBorins]

/cc @nodejs/ctc

[jbergstroem]

The likely cause is us switching www hosts (was running out of disk space). Not sure exactly what.

1. We moved the dist indexer to a new place: https://github.com/nodejs/nodejs-dist-indexer
2. The dist indexer at our old www host is pretty much identical to currently deployed at new place (just making sure that no un-committed code was running)
3. The scripts look like they are invoked the same way

[rvagg]

I've run it again and it's updated. This can happen if the tag is not pushed to the public repo before being promoted. @evanlucas I think you did 6.8.1, do you recall if this happened?

[evanlucas]

Looking back, I may not have pushed the tag when I promoted

[rvagg]

pretty sure nodejs/node#9142 will solve this