Skip to content

Update GitHub Actions to create and push the OCUDU container image#30

Merged
gab-arrobo merged 3 commits intoopennetworkinglab:mainfrom
gab-arrobo:update-gha
Mar 26, 2026
Merged

Update GitHub Actions to create and push the OCUDU container image#30
gab-arrobo merged 3 commits intoopennetworkinglab:mainfrom
gab-arrobo:update-gha

Conversation

@gab-arrobo
Copy link
Copy Markdown
Contributor

No description provided.

Signed-off-by: Arrobo, Gabriel <gabriel.arrobo@intel.com>
Signed-off-by: Arrobo, Gabriel <gabriel.arrobo@intel.com>
Signed-off-by: Arrobo, Gabriel <gabriel.arrobo@intel.com>
@@ -1 +1 @@
0.5.0
0.4.1-dev
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

It seems unusual to downgrade the version -- is this intentional?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I agree with you. The problem is that for some reason 0.4.0 was never published and that is why I am making this change

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@andybavier, what would you suggest?

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

I think the principle should be that published artifact numbers always go up. If 0.5.0 was published, then 0.5.1-dev. If 0.4.0 is the highest published version, then 0.4.1-dev should be fine.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Sorry, I meant that 0.4.0 and 0.5.0 were never published. However, we do have container images for 0.4.0. That is, after this PR is merged, I am going to open a PR for a minor release and create 0.5.0

Copy link
Copy Markdown

@andybavier andybavier left a comment

Choose a reason for hiding this comment

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

+1, assuming that v0.5.0 was not published

@gab-arrobo gab-arrobo merged commit 86633e3 into opennetworkinglab:main Mar 26, 2026
4 checks passed
@gab-arrobo gab-arrobo deleted the update-gha branch March 26, 2026 02:10
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.

2 participants