Skip to content

Invoke provisionator with AUTH_TOKEN_GITHUB_COM set#11041

Merged
rmarinho merged 1 commit intomainfrom
dev/cadsit/provisionator-use-auth-envvar
Nov 2, 2022
Merged

Invoke provisionator with AUTH_TOKEN_GITHUB_COM set#11041
rmarinho merged 1 commit intomainfrom
dev/cadsit/provisionator-use-auth-envvar

Conversation

@cadsit
Copy link
Copy Markdown
Contributor

@cadsit cadsit commented Nov 1, 2022

Description of Change

Per 1658536, we are updating all call sites of provisionator in CI to include setting AUTH_TOKEN_GITHUB_COM to allow us to support downloading from private blob containers through dl.internalx.com

@Eilon Eilon added the area-infrastructure CI, Maestro / Coherency, upstream dependencies/versions label Nov 1, 2022
@rmarinho rmarinho enabled auto-merge (squash) November 2, 2022 11:07
@rmarinho rmarinho merged commit 900d5f7 into main Nov 2, 2022
@rmarinho rmarinho deleted the dev/cadsit/provisionator-use-auth-envvar branch November 2, 2022 11:17
rmarinho pushed a commit that referenced this pull request Nov 2, 2022
# Conflicts:
#	eng/pipelines/common/provision.yml
rmarinho added a commit that referenced this pull request Nov 3, 2022
* Fix Author Name (#11015)

* Add await for color check to avoid crash (#11022)

* Remove unused reference (#11048)

* Invoke provisionator with AUTH_TOKEN_GITHUB_COM set (#11041)

# Conflicts:
#	eng/pipelines/common/provision.yml

Co-authored-by: Shane Neuville <shneuvil@microsoft.com>
Co-authored-by: Jonathan Dick <jondick@gmail.com>
Co-authored-by: Connor Adsit <connor.adsit@gmail.com>
@github-actions github-actions bot locked and limited conversation to collaborators Dec 16, 2023
@samhouts samhouts added the fixed-in-8.0.0-preview.1.7762 Look for this fix in 8.0.0-preview.1.7762! label Aug 2, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

area-infrastructure CI, Maestro / Coherency, upstream dependencies/versions fixed-in-8.0.0-preview.1.7762 Look for this fix in 8.0.0-preview.1.7762!

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants