[check-node-versions] Fails in 5 different ways

[HarshaNalluru]

I was releasing app-config and hit all sorts of errors with check-node-versions.

1. Codespaces
   npx dev-tool samples check-node-versions --artifact-path https://bit.ly/2Vj9AEo --node-versions "14,16" is my command
   

2. I gave up and started using the dev-tool in my local.
   

3. I tried deleting them with the container ids and it says they are not found. Tried again, this time it goes past that step and hits a new issue.
   

4. Samples generation also seem to have a problem.
   We are generating and publishing and never cared about running them. @witemple-msft
   

5. I don't even understand what I should be doing.
   I did give the artifact URL, I don;t know why it still fails.
   

Dev-tool with the check-node-versions used to sweetly run all the samples at once before, I just want to get that behavior back! 😥

[deyaaeldeen]

@HarshaNalluru I just found out that installing packages from URLs provided by the release pipeline does not actually work! because they're not direct links. See https://teams.microsoft.com/l/message/19:[email protected]/1627675696540?tenantId=72f988bf-86f1-41af-91ab-2d7cd011db47&groupId=3e17dcb0-4257-4a30-b843-77f47f1d4121&parentMessageId=1627675696540&teamName=Azure%20SDK&channelName=Engineering%20System%20%F0%9F%9B%A0%EF%B8%8F&createdTime=1627675696540. Could you try download the artifact first and then use the local path in the --artifact-path argument? I just tested on linux and it works fine.

[deyaaeldeen]

Ok, I also tested it on windows and it works fine for me:

I suspect something went bad with your docker some reason but I can not reproduce what you saw. Although it works for me on windows, could you try on linux instead?

[deyaaeldeen]

Actually, there is a bug I just fixed in #16686 and I think that was responsible for issue at step 3. Let's try again after that PR gets merged.

[ghost]

Hi @HarshaNalluru. Thank you for opening this issue and giving us the opportunity to assist. We believe that this has been addressed. If you feel that further discussion is needed, please add a comment with the text “/unresolve” to remove the “issue-addressed” label and continue the conversation.

[HarshaNalluru]

I can create individual issues for the rest and close this if you want @deyaaeldeen

[HarshaNalluru]

Closing in favor of #16713

[HarshaNalluru]

#15619 tracks 1