Skip to content

Pass Testflinger credentials as workflow inputs#67

Merged
farshidtz merged 4 commits into
v2from
tf-auth
Jul 10, 2026
Merged

Pass Testflinger credentials as workflow inputs#67
farshidtz merged 4 commits into
v2from
tf-auth

Conversation

@farshidtz

@farshidtz farshidtz commented Jul 9, 2026

Copy link
Copy Markdown
Member

Testflinger has started mandating authentication from the self-hosted runners.

Depends on:

@farshidtz farshidtz marked this pull request as ready for review July 9, 2026 15:02
Comment thread .github/workflows/reuse-pr-build-test.yaml Outdated

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the reusable PR build/test GitHub Actions workflow to support authenticated Testflinger runs now that authentication is required on the self-hosted runners.

Changes:

  • Adds Testflinger credentials as workflow_call parameters and forwards them into the called test-snap.yaml workflow.
  • Switches the called workflow ref from @v1 to @tf-auth to pick up the upstream authentication changes.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .github/workflows/reuse-pr-build-test.yaml Outdated
Comment thread .github/workflows/reuse-pr-build-test.yaml
Comment thread .github/workflows/reuse-pr-build-test.yaml

@jpm-canonical jpm-canonical left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@farshidtz farshidtz merged commit 5ddb49d into v2 Jul 10, 2026
1 check passed
@farshidtz farshidtz deleted the tf-auth branch July 10, 2026 10:25
@farshidtz farshidtz changed the title Add Testflinger credentials as workflow input secrets Pass Testflinger credentials as workflow inputs Jul 10, 2026
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