Skip to content

V8 Instance steps for multiple commands #3485

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

Merged
merged 2 commits into from
Apr 9, 2025
Merged

V8 Instance steps for multiple commands #3485

merged 2 commits into from
Apr 9, 2025

Conversation

Samze
Copy link
Contributor

@Samze Samze commented Apr 9, 2025

v8 version of #3400

joaopapereira and others added 2 commits April 9, 2025 11:33
* Fix error message for push command

Signed-off-by: João Pereira <joao.pereira@broadcom.com>

* Display current step in the canary deployment

Signed-off-by: João Pereira <joao.pereira@broadcom.com>

* Add --instance-steps to the restart command

Signed-off-by: João Pereira <joao.pereira@broadcom.com>

* Add flag --instance-steps to restage command

Signed-off-by: João Pereira <joao.pereira@broadcom.com>

* Add --instance-steps to rollback

Signed-off-by: João Pereira <joao.pereira@broadcom.com>

* Add --instance-steps to copy-source command

Signed-off-by: João Pereira <joao.pereira@broadcom.com>

* Fix rollback that was not setting the correct values on the deployment

Signed-off-by: João Pereira <joao.pereira@broadcom.com>

* Add min capi version check on instance steps commands

Signed-off-by: João Pereira <joao.pereira@broadcom.com>

* Fix pointer reference

* Add version to copy-source canary step test

* Update canary step version requirement

* Fix continue deployment test

* Fix CAPI versions in tests

* Fixes for canary integration tests

* Fix flakey canary test

* Add missing continue to copy-source canary test

---------

Signed-off-by: João Pereira <joao.pereira@broadcom.com>
Co-authored-by: Sam Gunaratne <385176+Samze@users.noreply.github.com>
Co-authored-by: Sam Gunaratne <sgunaratne@vmware.com>
Copy link
Member

@gururajsh gururajsh left a comment

Choose a reason for hiding this comment

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

LGTM

@Samze Samze merged commit 3d1f22a into v8 Apr 9, 2025
19 checks passed
@Samze Samze deleted the v8_steps_commands branch April 9, 2025 21:38
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.

3 participants