Skip to content

Bugfix/integration test#6

Merged
krakenhavoc merged 4 commits into
mainfrom
bugfix/integration-test
Apr 13, 2026
Merged

Bugfix/integration test#6
krakenhavoc merged 4 commits into
mainfrom
bugfix/integration-test

Conversation

@krakenhavoc

Copy link
Copy Markdown
Contributor

This pull request makes several important updates to the entrypoint.sh script for improved compatibility with the latest krakenkey CLI and better handling of environment variables. The changes focus on updating command-line argument styles, improving PATH management, and refining how commands are constructed and executed.

CLI Argument Style Updates:

  • Updated all krakenkey CLI invocations to use the new single-dash argument style (e.g., -domain instead of --domain) and reordered command arguments to match the latest CLI expectations.

Environment and PATH Handling:

  • Changed the method of updating the executable PATH from appending to GITHUB_PATH to exporting a new PATH variable directly, ensuring the CLI is available in the current shell session.

Command Construction Improvements:

  • Refactored command construction for issuing and renewing certificates to use conditional logic for boolean flags (e.g., only adding -auto-renew or -wait if enabled), and grouped arguments for clarity and correctness.
  • Updated the set_outputs() function to use the new CLI argument style and command ordering for the cert show command.

These changes ensure the script remains compatible with the evolving CLI and improve reliability in CI/CD environments.

sha256sum -c expects the file at the path listed in checksums.txt.
Downloading to cli.tar.gz caused a 'No such file or directory' error
during verification.
GITHUB_PATH changes only take effect in subsequent steps, not within
the same composite action step. Use export PATH instead.
Global flags (--api-url, --output, --no-color) must appear before the
subcommand. Subcommand flags use single-dash syntax (-domain, -wait).
Match the CLI's pflag migration — all subcommand flags now use
--double-dash syntax consistently.
@krakenhavoc
krakenhavoc merged commit 3d0b2d6 into main Apr 13, 2026
3 checks passed
@krakenhavoc
krakenhavoc deleted the bugfix/integration-test branch April 13, 2026 14:09
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.

1 participant