Skip to content

chore(build): fix build and push script to remove v prefix#467

Merged
kmova merged 1 commit intoopenebs-archive:masterfrom
akhilerm:remove-v-prefix
Aug 4, 2020
Merged

chore(build): fix build and push script to remove v prefix#467
kmova merged 1 commit intoopenebs-archive:masterfrom
akhilerm:remove-v-prefix

Conversation

@akhilerm
Copy link
Copy Markdown
Contributor

Signed-off-by: Akhil Mohan akhil.mohan@mayadata.io

Why is this PR required? What issue does it fix?:
Currently the ci images from release branches are having a prefix v, eg: v0.7.x-ci. But to make naming consistent across all the images we need to use only 0.7.x-ci.

What this PR does?:
This PR removes the prefix from both the push script as well as the multiarch image.

Does this PR require any upgrade changes?:
No

If the changes in this PR are manually verified, list down the scenarios covered::

Any additional information for your reviewer? :
This PR is part of #137 to remove v prefix from all image tags.

Checklist:

  • Fixes #
  • PR Title follows the convention of <type>(<scope>): <subject>
  • Has the change log section been updated?
  • Commit has unit tests
  • Commit has integration tests
  • (Optional) Are upgrade changes included in this PR? If not, mention the issue/PR to track:
  • (Optional) If documentation changes are required, which issue on https://github.com/openebs/openebs-docs is used to track them:

Currently the ci images from release branches are having a prefix `v`,
eg: `v0.7.x-ci`. But to make naming consistent across all the images
we need to use only `0.7.x-ci`. This PR removes the prefix from both
the push script as well as the multiarch image.

Signed-off-by: Akhil Mohan <akhil.mohan@mayadata.io>
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Jul 31, 2020

Codecov Report

Merging #467 into master will increase coverage by 0.21%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #467      +/-   ##
==========================================
+ Coverage   42.68%   42.90%   +0.21%     
==========================================
  Files          71       71              
  Lines        3228     3228              
==========================================
+ Hits         1378     1385       +7     
+ Misses       1740     1735       -5     
+ Partials      110      108       -2     
Impacted Files Coverage Δ
cmd/ndm_daemonset/probe/udevprobe.go 59.74% <0.00%> (+2.59%) ⬆️
cmd/ndm_daemonset/probe/eventhandler.go 48.97% <0.00%> (+6.12%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update eb668ed...e18ab45. Read the comment docs.

Copy link
Copy Markdown
Contributor

@xunholy xunholy 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 bc0a40d into openebs-archive:master Aug 4, 2020
@akhilerm akhilerm deleted the remove-v-prefix branch December 13, 2021 05:51
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.

5 participants