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

chore(push_script): refactor makefile and push scirpt #59

Merged
merged 3 commits into from
May 7, 2020

Conversation

sonasingh46
Copy link
Contributor

@sonasingh46 sonasingh46 commented May 7, 2020

This PR does the following :

  1. The container images should be pushed without a v in the tag SEMVER.
    The refactors script to trims the leading v before pushing

  2. Change the env variable HUB_USER to IMAGE_ORG for consistency across openebs repos.
    This variable holds the image repository user name.

  3. Modifies the Dockerfiles to add the required labels.

Signed-off-by: Ashutosh Kumar ashutosh.kumar@mayadata.io

Signed-off-by: Ashutosh Kumar <ashutosh.kumar@mayadata.io>
@sonasingh46 sonasingh46 requested a review from kmova May 7, 2020 11:55
@sonasingh46 sonasingh46 self-assigned this May 7, 2020
Signed-off-by: Ashutosh Kumar <ashutosh.kumar@mayadata.io>
@sonasingh46 sonasingh46 changed the title chore(push_script): trim leading v from image tag chore(push_script): refactor makefile and push scirpt May 7, 2020
@sonasingh46 sonasingh46 changed the title chore(push_script): refactor makefile and push scirpt [WIP]chore(push_script): refactor makefile and push scirpt May 7, 2020
build/push Outdated Show resolved Hide resolved
build/push Outdated Show resolved Hide resolved
@sonasingh46 sonasingh46 force-pushed the pushscript_patch branch 3 times, most recently from 7776b13 to b8b739e Compare May 7, 2020 16:19
@sonasingh46 sonasingh46 changed the title [WIP]chore(push_script): refactor makefile and push scirpt chore(push_script): refactor makefile and push scirpt May 7, 2020
Signed-off-by: Ashutosh Kumar <ashutosh.kumar@mayadata.io>
Copy link
Contributor

@prateekpandey14 prateekpandey14 left a comment

Choose a reason for hiding this comment

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

lgtm

@kmova kmova merged commit 9a0149f into openebs-archive:master May 7, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants