Skip to content

Allow inclusion of repository in the tag name, so ghcr and dockerub c…#345

Merged
sergio-mena merged 9 commits intov0.34.xfrom
lasarojc/registry-in-tag
Feb 21, 2023
Merged

Allow inclusion of repository in the tag name, so ghcr and dockerub c…#345
sergio-mena merged 9 commits intov0.34.xfrom
lasarojc/registry-in-tag

Conversation

@lasarojc
Copy link
Contributor

…an be used, including in the same experiment

Should be ported to 0.37 and main.


PR checklist

  • Tests written/updated
  • Changelog entry added in .changelog (we use
    unclog to manage our changelog)
  • Updated relevant documentation (docs/ or spec/) and code comments

…an be used, including in the same experiment
@lasarojc lasarojc requested a review from a team as a code owner February 14, 2023 17:27
@lasarojc lasarojc self-assigned this Feb 14, 2023
@codecov
Copy link

codecov bot commented Feb 14, 2023

Codecov Report

Base: 48.40% // Head: 48.37% // Decreases project coverage by -0.04% ⚠️

Coverage data is based on head (25d7435) compared to base (38abebd).
Patch coverage: 0.00% of modified lines in pull request are covered.

❗ Current head 25d7435 differs from pull request most recent head 9431984. Consider uploading reports for the commit 9431984 to get more accurate results

Additional details and impacted files
@@             Coverage Diff             @@
##           v0.34.x     #345      +/-   ##
===========================================
- Coverage    48.40%   48.37%   -0.04%     
===========================================
  Files          280      280              
  Lines        49887    49908      +21     
===========================================
- Hits         24150    24145       -5     
- Misses       23878    23900      +22     
- Partials      1859     1863       +4     
Impacted Files Coverage Δ
test/e2e/generator/generate.go 9.18% <0.00%> (-0.71%) ⬇️
test/e2e/generator/main.go 0.00% <0.00%> (ø)
privval/signer_listener_endpoint.go 81.74% <0.00%> (-9.53%) ⬇️
privval/socket_listeners.go 79.72% <0.00%> (-4.06%) ⬇️
privval/signer_endpoint.go 74.72% <0.00%> (-2.20%) ⬇️
consensus/reactor.go 78.38% <0.00%> (-1.84%) ⬇️
statesync/syncer.go 75.07% <0.00%> (-0.58%) ⬇️
blockchain/v0/reactor.go 76.02% <0.00%> (-0.35%) ⬇️
consensus/state.go 70.30% <0.00%> (-0.25%) ⬇️
light/client.go 72.64% <0.00%> (-0.15%) ⬇️
... and 9 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

Copy link
Collaborator

@sergio-mena sergio-mena left a comment

Choose a reason for hiding this comment

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

Thanks for this!

@sergio-mena sergio-mena merged commit 5bdaf56 into v0.34.x Feb 21, 2023
@sergio-mena sergio-mena deleted the lasarojc/registry-in-tag branch February 21, 2023 10:06
lasarojc added a commit that referenced this pull request Feb 21, 2023
#345)

* Allow inclusion of repository in the tag name, so ghcr and dockerub can be used, including in the same experiment

* Moves UpgradeVersion to Node, so nodes can be upgraded to different versions. Uses multiversion to set UpgradeVersion.

* Updates the path for the local-version image.

* Do not use yoda conditions.

* Use the right code coverage definitions

* Revert use of . We will only do a maximum of 2 versions per run
lasarojc added a commit that referenced this pull request Feb 21, 2023
#345)

* Allow inclusion of repository in the tag name, so ghcr and dockerub can be used, including in the same experiment

* Moves UpgradeVersion to Node, so nodes can be upgraded to different versions. Uses multiversion to set UpgradeVersion.

* Updates the path for the local-version image.

* Do not use yoda conditions.

* Use the right code coverage definitions

* Revert use of . We will only do a maximum of 2 versions per run
lasarojc added a commit that referenced this pull request Feb 21, 2023
#345) (#362)

* Allow inclusion of repository in the tag name, so ghcr and dockerub can be used, including in the same experiment

* Moves UpgradeVersion to Node, so nodes can be upgraded to different versions. Uses multiversion to set UpgradeVersion.

* Updates the path for the local-version image.

* Do not use yoda conditions.

* Use the right code coverage definitions

* Revert use of . We will only do a maximum of 2 versions per run
mergify bot pushed a commit that referenced this pull request Feb 21, 2023
#345) (#362)

* Allow inclusion of repository in the tag name, so ghcr and dockerub can be used, including in the same experiment

* Moves UpgradeVersion to Node, so nodes can be upgraded to different versions. Uses multiversion to set UpgradeVersion.

* Updates the path for the local-version image.

* Do not use yoda conditions.

* Use the right code coverage definitions

* Revert use of . We will only do a maximum of 2 versions per run

(cherry picked from commit 45a08f3)
lasarojc added a commit that referenced this pull request Feb 21, 2023
#345) (#362) (#363)

* Allow inclusion of repository in the tag name, so ghcr and dockerub can be used, including in the same experiment

* Moves UpgradeVersion to Node, so nodes can be upgraded to different versions. Uses multiversion to set UpgradeVersion.

* Updates the path for the local-version image.

* Do not use yoda conditions.

* Use the right code coverage definitions

* Revert use of . We will only do a maximum of 2 versions per run

(cherry picked from commit 45a08f3)

Co-authored-by: Lasaro <lasaro@informal.systems>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

No open projects
Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants