Skip to content

refactor(submit): modify phase polling to match CLI#944

Merged
rene-oromtz merged 4 commits intomainfrom
refactor/step-per-phase
Mar 4, 2026
Merged

refactor(submit): modify phase polling to match CLI#944
rene-oromtz merged 4 commits intomainfrom
refactor/step-per-phase

Conversation

@rene-oromtz
Copy link
Copy Markdown
Contributor

@rene-oromtz rene-oromtz commented Feb 27, 2026

Description

Since the CLI can now poll per individual phase, this PR aligns the submit action to this per phase behavior.
With this change each phase is a single step so its a bit more visible on the CI run as a dropdown.

Resolved issues

Resolves CERTTF-822

Took the liberty to sneak the minor modification needed on the README to include the recently added queue-status action.
Resolves #938
Resolves CERTTF-841

Documentation

Web service API changes

Tests

Test run with all provision, test and reserve phases in job: https://github.com/canonical/tf-submit-action-test/actions/runs/22504017553

@rene-oromtz rene-oromtz requested a review from ajzobro February 27, 2026 21:54
Comment thread .github/actions/submit/action.yaml Outdated
@rene-oromtz rene-oromtz requested a review from ajzobro March 3, 2026 16:41
@rene-oromtz rene-oromtz force-pushed the refactor/step-per-phase branch from 1dad6f4 to dfe6118 Compare March 3, 2026 16:56
Copy link
Copy Markdown
Collaborator

@ajzobro ajzobro left a comment

Choose a reason for hiding this comment

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

LGTM

@rene-oromtz rene-oromtz merged commit 5d968bf into main Mar 4, 2026
2 checks passed
@rene-oromtz rene-oromtz deleted the refactor/step-per-phase branch March 4, 2026 19:48
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.

queue-status action not mentioned in README.md under github actions

2 participants