potiuk commented on a change in pull request #10368:
URL: https://github.com/apache/airflow/pull/10368#discussion_r473633829
##########
File path: scripts/ci/libraries/_initialization.sh
##########
@@ -223,94 +275,190 @@ function initialize_common_environment {
export CI_BUILD_ID="0"
}
+function initialize_git_variables() {
+ # SHA of the commit for the current sources
+ COMMIT_SHA="$(git rev-parse HEAD || echo "Unknown")"
+ export COMMIT_SHA
+}
+
+function initialize_github_variables() {
+ # Defaults for interacting with GitHub
+ export GITHUB_REPOSITORY=${GITHUB_REPOSITORY:="apache/airflow"}
+ export GITHUB_REGISTRY=${GITHUB_REGISTRY:="docker.pkg.github.com"}
+ export USE_GITHUB_REGISTRY=${USE_GITHUB_REGISTRY:="false"}
+ export
GITHUB_REGISTRY_PULL_IMAGE_TAG=${GITHUB_REGISTRY_PULL_IMAGE_TAG:="latest"}
+ export
GITHUB_REGISTRY_PUSH_IMAGE_TAG=${GITHUB_REGISTRY_PUSH_IMAGE_TAG:="latest"}
+
+}
+
+# Common environment that is initialized by both Breeze and CI scripts
+function initialize_common_environment() {
Review comment:
Yeah. And unit testable. Soon.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
