Skip to content

Allow inclusion of repository in the tag name, so ghcr and dockerub ...#362

Merged
lasarojc merged 1 commit intomainfrom
lasarojc/main/registry-in-tag
Feb 21, 2023
Merged

Allow inclusion of repository in the tag name, so ghcr and dockerub ...#362
lasarojc merged 1 commit intomainfrom
lasarojc/main/registry-in-tag

Conversation

@lasarojc
Copy link
Contributor

@lasarojc lasarojc commented Feb 21, 2023

can be used, inclusively in the same experiment.

This is a forward port of (#345)


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

#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 lasarojc requested a review from a team as a code owner February 21, 2023 10:56
@lasarojc lasarojc self-assigned this Feb 21, 2023
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!

@lasarojc lasarojc merged commit 45a08f3 into main Feb 21, 2023
@lasarojc lasarojc deleted the lasarojc/main/registry-in-tag branch February 21, 2023 11:52
@lasarojc lasarojc restored the lasarojc/main/registry-in-tag branch February 21, 2023 11:52
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
Copy link
Contributor Author

@Mergifyio refresh

@mergify
Copy link
Contributor

mergify bot commented Feb 21, 2023

refresh

✅ Pull request refreshed

@lasarojc lasarojc deleted the lasarojc/main/registry-in-tag branch February 21, 2023 11:54
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