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

Add a GitHub action based release job #723

Merged
merged 3 commits into from
Dec 11, 2020
Merged

Add a GitHub action based release job #723

merged 3 commits into from
Dec 11, 2020

Conversation

ericwill
Copy link
Contributor

@ericwill ericwill commented Dec 1, 2020

Signed-off-by: Eric Williams ericwill@redhat.com

What does this PR do?

Calls make-release.sh with the necessary parameters to release the che-plugin-registry without using centos-ci. Added build/publish steps to the make-release.sh script.

Screenshot/screencast of this PR

N/A

What issues does this PR fix or reference?

Part of eclipse-che/che#18072

How to test this PR?

N/A

PR Checklist

As the author of this Pull Request I made sure that:

Reviewers

Reviewers, please comment how you tested the PR when approving it.

Copy link
Contributor

@benoitf benoitf left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have not tested but LGTM

make-release.sh Outdated Show resolved Hide resolved
make-release.sh Outdated Show resolved Hide resolved
Part of eclipse-che/che#18072

Signed-off-by: Eric Williams <ericwill@redhat.com>
Signed-off-by: Eric Williams <ericwill@redhat.com>
Signed-off-by: Eric Williams <ericwill@redhat.com>
@ericwill
Copy link
Contributor Author

@benoitf do you have the secrets to log into Dockerhub?

@benoitf
Copy link
Contributor

benoitf commented Dec 10, 2020

yes I think I can get some

@benoitf
Copy link
Contributor

benoitf commented Dec 10, 2020

I can register them tomorrow if it works for you

@ericwill
Copy link
Contributor Author

I can register them tomorrow if it works for you

Any time before next Wednesday's release works fine, thanks! Also please for the devfile registry, as there is the same PR there.

@benoitf
Copy link
Contributor

benoitf commented Dec 11, 2020

I forgot to say but I added the docker.io credentials this morning (my morning :-)

@ericwill ericwill merged commit 2867bf3 into eclipse-che:master Dec 11, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants