Skip to content

ci: fix vTDS deploy create-github-app-token GitHub Action step. - #77

Merged
synackd merged 1 commit into
OpenCHAMI:mainfrom
davidfluck-hpe:fix-vtds-deploy
Sep 3, 2026
Merged

synackd merged 1 commit into
OpenCHAMI:mainfrom
davidfluck-hpe:fix-vtds-deploy

Conversation

@davidfluck-hpe

Copy link
Copy Markdown
Contributor

Description

This changeset fixes a bug where the create-github-app-token GitHub Action step output was not being referenced correctly, leading to subsequent empty token variables, which caused the deploy action to fail.

Checklist

  • My code follows the style guidelines of this project
  • I have added/updated comments where needed
  • I have added tests that prove my fix is effective or my feature works
  • I have run make test (or equivalent) locally and all tests pass
  • I have updated the relevant documentation (CLI examples, man pages, README, other docs, etc.)
  • DCO Sign-off: All commits are signed off (git commit -s) with my real name and email
  • REUSE Compliance:
    • Each new/modified source file has SPDX copyright and license headers
    • Any non-commentable files include a <filename>.license sidecar
    • All referenced licenses are present in the LICENSES/ directory

Type of Change

  • Bug fix
  • New feature
  • Breaking change
  • Documentation update
  • Dependency update
  • Build system/CI

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

Changes look fine, just need DCO on commit.

Signed-off-by: David Fluck <david.fluck@hpe.com>
@davidfluck-hpe

Copy link
Copy Markdown
Contributor Author

Gah, I'm sorry, I always get GitHub verified commits mixed up with signed commits heh. Pushed.

@synackd synackd 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. No worries, it can be confusing.

@synackd synackd changed the title Fix vTDS deploy create-github-app-token GitHub Action step. ci: fix vTDS deploy create-github-app-token GitHub Action step. Sep 3, 2026
@synackd
synackd merged commit b6ac727 into OpenCHAMI:main Sep 3, 2026
2 checks passed
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