Skip to content

Conversation

joaopapereira
Copy link
Contributor

Description of the Change

This PR tests will not pass until we have a version of CAPI that supports the canary weights

This PR enables the command cf app to display the current and total number of steps in the canary deployment

@joaopapereira joaopapereira force-pushed the main-discover-canary-steps-status branch from c77713f to fbb669a Compare February 11, 2025 17:25
@joaopapereira joaopapereira changed the title Discover canary steps status [main] Discover canary steps status Feb 12, 2025
@joaopapereira joaopapereira force-pushed the main-discover-canary-steps-status branch 3 times, most recently from ffdfa4c to 2e14154 Compare February 19, 2025 17:26
Signed-off-by: João Pereira <joao.pereira@broadcom.com>
Signed-off-by: João Pereira <joao.pereira@broadcom.com>
@joaopapereira joaopapereira force-pushed the main-discover-canary-steps-status branch from 2e14154 to 4a99027 Compare February 19, 2025 21:07
@Samze Samze merged commit 18d4277 into cloudfoundry:main Mar 20, 2025
18 of 19 checks passed
Samze added a commit that referenced this pull request Mar 20, 2025
* 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>

* Fix canary steps tests

* Fix continue test instance-step command

* Fix integration test for canary

---------

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>
Samze added a commit that referenced this pull request Mar 20, 2025
* 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>

* Fix canary steps tests

* Fix continue test instance-step command

* Fix integration test for canary

---------

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>
a-b pushed a commit that referenced this pull request Mar 20, 2025
* Fix error message for push command



* Display current step in the canary deployment



* Fix canary steps tests

* Fix continue test instance-step command

* Fix integration test for canary

---------

Signed-off-by: João Pereira <joao.pereira@broadcom.com>
Co-authored-by: João Pereira <joao.pereira@broadcom.com>
Copy link
Member

@a-b a-b left a comment

Choose a reason for hiding this comment

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

LGTM

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