Skip to content

refactor(ci): move integration-test common steps into composite action - #7070

Open
blackboxsw wants to merge 1 commit into
canonical:mainfrom
blackboxsw:refactor-composite-action
Open

refactor(ci): move integration-test common steps into composite action#7070
blackboxsw wants to merge 1 commit into
canonical:mainfrom
blackboxsw:refactor-composite-action

Conversation

@blackboxsw

Copy link
Copy Markdown
Collaborator

Propsed commit message

refactor(ci): move integration-test common steps into composite action

Replace the reusable workflow .github/workflows/100-dispatch-common.yml
with a composite action .github/actions/integration-test containing
shared integration-test steps (secret assertion, checkout, LXD/SSH/
pycloudlib setup, tox run, report publish, cleanup).

This allows platform-specific secret changes and pre-flight workflow
steps such as credential file setup in subsequent workflows avoiding
pulling in platform-specific setup into the common integration testing.

Behavior is preseved. Update all existing platform workflows to now
pass in environment variables to the composite action the boundary
from workflow to composite action doesn't allow passing workflow
`secrets`.

Additionally, to run composite actions a preliminary checkout step is
required in the scheduled workflow to provide visibility to the
.github/actions dir.

Additional Context

Split out from #6969

Test Steps

Merge type

  • Squash merge using "Proposed Commit Message"
  • Rebase and merge unique commits. Requires commit messages per-commit each referencing the pull request number (#<PR_NUM>)

Replace the reusable workflow .github/workflows/100-dispatch-common.yml
with a composite action .github/actions/integration-test containing
shared integration-test steps (secret assertion, checkout, LXD/SSH/
pycloudlib setup, tox run, report publish, cleanup).

This allows platform-specific secret changes and pre-flight workflow
steps such as credential file setup in subsequent workflows avoiding
pulling in platform-specific setup into the common integration testing.

Behavior is preseved. Update all existing platform workflows to now
pass in environment variables to the composite action the boundary
from workflow to composite action doesn't allow passing workflow
`secrets`.

Additionally, to run composite actions a preliminary checkout step is
required in the scheduled workflow to provide visibility to the
.github/actions dir.
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