Skip to content

V8 Instance steps for multiple commands#3485

Merged
Samze merged 2 commits into
v8from
v8_steps_commands
Apr 9, 2025
Merged

V8 Instance steps for multiple commands#3485
Samze merged 2 commits into
v8from
v8_steps_commands

Conversation

@Samze

@Samze Samze commented Apr 9, 2025

Copy link
Copy Markdown
Contributor

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>

@gururajsh gururajsh left a comment

Copy link
Copy Markdown
Member

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
@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