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

Publish a versioned che-plugin-registry release to GitHub pages and surge.sh #19268

Closed
ericwill opened this issue Mar 10, 2021 · 6 comments · Fixed by eclipse-che/che-plugin-registry#922
Assignees
Labels
area/ci CI build and releases, PR testing, & whitelabel/productization issues area/plugin-registry kind/task Internal things, technical debt, and to-do tasks to be performed. new&noteworthy For new and/or noteworthy issues that deserve a blog post, new docs, or emphasis in release notes severity/P2 Has a minor but important impact to the usage or development of the system. sprint/current sprint/next
Milestone

Comments

@ericwill
Copy link
Contributor

Is your task related to a problem? Please describe.

Currently, we are only publishing a nightly che-plugin-registry release on surge.sh.

Describe the solution you'd like

It would be nice to publish a version at release time, for example:

and so on. This way we can reference versioned plugin registry releases in other places, like the devfile registry.

Describe alternatives you've considered

Publish to GitHub pages instead of surge.sh.

Additional context

Needed in order to enable community plugins and devfiles on Dev Sandbox.

@ericwill ericwill added kind/task Internal things, technical debt, and to-do tasks to be performed. area/plugin-registry area/ci CI build and releases, PR testing, & whitelabel/productization issues labels Mar 10, 2021
@che-bot che-bot added the status/need-triage An issue that needs to be prioritized by the curator responsible for the triage. See https://github. label Mar 10, 2021
@ericwill ericwill removed the status/need-triage An issue that needs to be prioritized by the curator responsible for the triage. See https://github. label Mar 10, 2021
@davidfestal davidfestal added the severity/P2 Has a minor but important impact to the usage or development of the system. label Mar 11, 2021
@ericwill ericwill mentioned this issue Mar 18, 2021
46 tasks
@ericwill ericwill self-assigned this Mar 24, 2021
@ericwill ericwill added this to the 7.29 milestone Mar 24, 2021
@sunix
Copy link
Contributor

sunix commented Apr 6, 2021

Currently looking at it.
Not sure we can use

on:
  workflow_run:
    workflows: ["Release Che Plugin Registry"]
    types:
      - completed

to publish after a release job.
However it looks like the image is pushed before the tag creation. See https://github.com/eclipse-che/che-plugin-registry/blob/master/make-release.sh#L34 and https://github.com/eclipse-che/che-plugin-registry/blob/master/make-release.sh#L152
So we could trigger the publishing on a tag or release event

@ericwill
Copy link
Contributor Author

ericwill commented Apr 6, 2021

Currently looking at it.
Not sure we can use

on:
  workflow_run:
    workflows: ["Release Che Plugin Registry"]
    types:
      - completed

to publish after a release job.
However it looks like the image is pushed before the tag creation. See https://github.com/eclipse-che/che-plugin-registry/blob/master/make-release.sh#L34 and https://github.com/eclipse-che/che-plugin-registry/blob/master/make-release.sh#L152
So we could trigger the publishing on a tag or release event

Could we not add the surge.sh publish step inside the make-release.sh script?

@sunix
Copy link
Contributor

sunix commented Apr 6, 2021

We could. It would required installing surge in make-release.sh. maybe easier in a step following the release ? https://github.com/eclipse-che/che-plugin-registry/blob/master/.github/workflows/release.yml#L85-L92
Question is: do we want to mark the release job as failed if something is wrong with surge.sh ?

@ericwill
Copy link
Contributor Author

ericwill commented Apr 6, 2021

Question is: do we want to mark the release job as failed if something is wrong with surge.sh ?

Since we plan to start using it in other places (i.e. devfiles) we should probably make sure it runs successfully, yes. IMO this means it that the release should fail if the surge.sh publish fails.

Side note: we do not need to only publish to surge.sh, it might be a good idea to publish to GitHub pages as well.

@ericwill ericwill assigned sunix and unassigned ericwill Apr 6, 2021
@sunix
Copy link
Contributor

sunix commented Apr 6, 2021

if we want to use in other places, better not relying on surge.sh. It has been down couple of times. Github pages sounds like a good option.

@ericwill
Copy link
Contributor Author

ericwill commented Apr 6, 2021

if we want to use in other places, better not relying on surge.sh. It has been down couple of times. Github pages sounds like a good option.

Sounds good! 👍

@ericwill ericwill mentioned this issue Apr 8, 2021
42 tasks
@l0rd l0rd changed the title Publish a versioned che-plugin-registry release to surge.sh Publish a versioned che-plugin-registry release to GitHub pages and surge.sh Apr 20, 2021
@l0rd l0rd added the new&noteworthy For new and/or noteworthy issues that deserve a blog post, new docs, or emphasis in release notes label Apr 20, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/ci CI build and releases, PR testing, & whitelabel/productization issues area/plugin-registry kind/task Internal things, technical debt, and to-do tasks to be performed. new&noteworthy For new and/or noteworthy issues that deserve a blog post, new docs, or emphasis in release notes severity/P2 Has a minor but important impact to the usage or development of the system. sprint/current sprint/next
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants