Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[plugin registry] Do not reference marketplace.visualstudio.com in plugin registry meta.yaml #14573

Closed
2 of 3 tasks
nickboldt opened this issue Sep 17, 2019 · 6 comments
Closed
2 of 3 tasks
Assignees
Labels
area/plugin-registry severity/P1 Has a major impact to usage or development of the system.
Milestone

Comments

@nickboldt
Copy link
Contributor

nickboldt commented Sep 17, 2019

Describe the bug

Microsoft says marketplace.visualstudio.com is for VS Code users ONLY.

Therefore we can't point from the plugin registry to marketplace.visualstudio.com anymore.

Example:

https://github.com/eclipse/che-plugin-registry/blob/master/v3/plugins/ms-vscode/node-debug/1.32.1/meta.yaml (last line)
https://github.com/eclipse/che-plugin-registry/blob/master/v3/plugins/ms-vscode/node-debug/latest/meta.yaml#L20

Che version

  • latest
  • nightly
  • other: please specify

Expected behavior

We should instead either:

  • pre-pull the artifacts and host them in the plugin registry container, or
  • host the artifacts on a redhat.com server (or download.jboss.org), or
  • pull files from github releases

And:

  • validate the meta.yaml files don't container any refs to marketplace.visualstudio.com (new test script)

Affected files

./v3/plugins/joaompinto/vscode-asciidoctor/2.7.6/meta.yaml 
./v3/plugins/joaompinto/vscode-asciidoctor/latest/meta.yaml 
./v3/plugins/ms-python/python/2019.2.5433/meta.yaml << fixed
./v3/plugins/ms-vscode/node-debug/1.32.1/meta.yaml 
./v3/plugins/ms-vscode/node-debug/1.35.2/meta.yaml 
./v3/plugins/ms-vscode/node-debug2/1.31.6/meta.yaml 
./v3/plugins/ms-vscode/node-debug2/1.33.0/meta.yaml 
./v3/plugins/ms-vscode/node-debug2/latest/meta.yaml 
./v3/plugins/ms-vscode/node-debug/latest/meta.yaml 

cc: @gorkem @slemeur @benoitf

@nickboldt nickboldt added area/plugin-registry severity/P1 Has a major impact to usage or development of the system. labels Sep 17, 2019
@nickboldt nickboldt self-assigned this Sep 17, 2019
@nickboldt
Copy link
Contributor Author

nickboldt commented Sep 18, 2019

joaompinto/vscode-asciidoctor does not publish its built vsix file to github, but only to marketplace.visualstudio.com -- opened asciidoctor/asciidoctor-vscode#221

Update: as of 2.7.7 vsix is now included in GH:

https://github.com/asciidoctor/asciidoctor-vscode/releases/tag/v2.7.7

@nickboldt
Copy link
Contributor Author

nickboldt commented Sep 20, 2019

node-debug and node-debug2 projects don't publish vsix files to github either.

Since we need an offline/airgap plugin registry solution anyway, and since I have to pull sources to make our Open Source Lawyers happy, perhaps it'd be worth rearchitecting the plugin registry build process so that it:

@mkuznyetsov
Copy link
Contributor

ATM I have Jenkins jobs that would perform a packaging of all desired extension .vsix files into tarball

Now it needs to replace the links to the relative location of extensions and provide those extensions from the tarball mentioned above.
This will be done via one of sh scripts for building plugin-registry, which I'm working on right now. I expect it would take 1-2 more days for a PR to review.
Then I expect there still will be a need to make sure about the whole build process, as well as how the tarball could be stored outside of Jenkins build job

@nickboldt
Copy link
Contributor Author

nickboldt commented Oct 18, 2019

Just got confirmation from Stevan that since the old node-debug extension is ONLY for Node <8 and the node-debug2 one is for Node 8+, we don't need to worry about building the old one for CRW 2.

Still trying to convince him that we should also remove the old one (Node 6 debug support? really?) from Che 7.4's plugin registry. See #14927

@nickboldt
Copy link
Contributor Author

Making progress in https://docs.google.com/spreadsheets/d/1TllnofYIlA7yTn7hIa8-LojGpHR1ACk1hVP-7fg_wsY/edit#gid=1935159288 but still not done. Slip to 7.7.0

@nickboldt
Copy link
Contributor Author

Followup work for CRW 2.1 (based on Che 7.9) to be done in https://issues.redhat.com/browse/CRW-578.

We can close this tracker once #14677 is complete

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/plugin-registry severity/P1 Has a major impact to usage or development of the system.
Projects
None yet
Development

No branches or pull requests

3 participants