feat: add optional Galaxy server auth inputs to sanity, unit, and integration workflows#103
Open
djdanielsson wants to merge 1 commit intoansible:mainfrom
Open
feat: add optional Galaxy server auth inputs to sanity, unit, and integration workflows#103djdanielsson wants to merge 1 commit intoansible:mainfrom
djdanielsson wants to merge 1 commit intoansible:mainfrom
Conversation
…egration workflows Collections that depend on certified content hosted on Automation Hub (or other private Galaxy servers) cannot install their dependencies without authentication. This adds three optional parameters to the sanity, unit, and integration reusable workflows: - `galaxy_server_url` (input) – URL of the additional Galaxy server - `galaxy_server_auth_url` (input) – SSO/token endpoint for the server - `galaxy_server_token` (secret) – authentication token When `galaxy_server_url` is provided, the ANSIBLE_GALAXY_SERVER_LIST env var is set to "certified,galaxy", and the corresponding ANSIBLE_GALAXY_SERVER_CERTIFIED_URL, _TOKEN, and _AUTH_URL env vars are populated from the inputs/secret. No temporary files are created. All three parameters are optional and fully backward-compatible; callers that do not pass them see no change in behavior. Made-with: Cursor
64924ef to
bed8e0d
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
galaxy_server_url,galaxy_server_auth_urlinputs andgalaxy_server_tokensecret) to the sanity, unit, and integration reusable workflows.galaxy_server_urlis provided,ANSIBLE_GALAXY_SERVER_LISTis set tocertified,galaxyand the correspondingANSIBLE_GALAXY_SERVER_CERTIFIED_URL,_TOKEN, and_AUTH_URLenv vars are populated on the tox run step. No temporary files are created.Motivation
Collections that depend on certified content hosted on Automation Hub (or other private Galaxy servers) currently cannot use these reusable workflows because
ansible-galaxy/ade installfails to resolve dependencies that require authentication. This forces downstream repos to inline the entire sanity/unit/integration workflow just to inject authentication.With this change, callers can simply pass the server URL and token:
Test plan
Made-with: Cursor