launchpad-reviewers team mailing list archive

[Quentin Debhi <mp+480171@xxxxxxxxxxxxxxxxxx>]

i did not check a specific MP, rather just read the code
i will add a relevant description with an explanation

Diff comments:

> diff --git a/lib/lp/services/config/schema-lazr.conf b/lib/lp/services/config/schema-lazr.conf
> index f634d11..45d17e8 100644
> --- a/lib/lp/services/config/schema-lazr.conf
> +++ b/lib/lp/services/config/schema-lazr.conf
> @@ -301,6 +301,20 @@ secrets: none
>  scan_malware: False
>  
>  
> +[craftbuild.soss]
> +# value is a JSON Object
> +# example:
> +# environment_variables: {
> +#    "CARGO_ARTIFACTORY1_URL": "https://canonical.example.com/artifactory/api/cargo/cargo-upstream/index/";,
> +#    "CARGO_ARTIFACTORY1_READ_AUTH": "%(read_auth)s",
> +#    "CARGO_REGISTRY_GLOBAL_CREDENTIAL_PROVIDERS": "cargo:token",
> +#    "MAVEN_ARTIFACTORY1_URL": "https://canonical.example.com/artifactory/api/maven/maven-upstream/index";,
> +#    "MAVEN_ARTIFACTORY1_READ_AUTH": "%(read_auth)s",

this is a placeholder, just demonstrating we could have multiple ones indeed, not necessarily with a number
idk yet how to create such variables inside of LP, but i think we have to ask a SRE for that

> +# }
> +# 
> +environment_variables: none 
> +
> +
>  [codebrowse]
>  # Where to store codebrowse's sqlite "files changed" caches.  If
>  # empty, the logs will be stored in a folder called 'logs' next to the


-- 
https://code.launchpad.net/~ruinedyourlife/launchpad/+git/launchpad/+merge/480171
Your team Launchpad code reviewers is requested to review the proposed merge of ~ruinedyourlife/launchpad:artifactory-env-var-for-sourcecraft-builds into launchpad:master.