Fixes identified in ecp facilities hackathon fixes:

- improve mirror git repo management
- minio s3 implementation needs endpoint_url
- Remove image key from rebuild-index job
- Remove image, rely on tags instead
This commit is contained in:
Scott Wittenburg 2019-09-03 11:45:49 -06:00 committed by Todd Gamblin
parent 6d745a56fd
commit 1050fa5546
4 changed files with 21 additions and 33 deletions

View file

@ -1,11 +1,8 @@
generate ci jobs: generate ci jobs:
variables:
git_strategy: clone
script: script:
- "./bin/generate-gitlab-ci-yml.sh" - "./bin/generate-gitlab-ci-yml.sh"
tags: tags:
- "spack-k8s" - "spack-pre-ci"
image: "scottwittenburg/spack_ci_generator_alpine"
artifacts: artifacts:
paths: paths:
- ci-generation - ci-generation

View file

@ -67,33 +67,25 @@ if [[ $? -ne 0 ]]; then
exit 1 exit 1
fi fi
cd "$original_directory"
mv .git "$temp_dir/original-git-dir"
git init .
git config user.email "robot@spack.io"
git config user.name "Spack Build Bot"
cp ${gen_ci_file} "${original_directory}/.gitlab-ci.yml" cp ${gen_ci_file} "${original_directory}/.gitlab-ci.yml"
git add .
# Remove global from here, it's clobbering people git identity config
git config --global user.email "robot@spack.io"
git config --global user.name "Build Robot"
commit_msg="Auto-generated commit testing ${current_branch} (${CI_COMMIT_SHA})"
cd ${original_directory}
echo "git status"
git status
echo "git branch"
git branch -D ___multi_ci___ 2> /dev/null || true
echo "git checkout"
git checkout -b ___multi_ci___
echo "git add"
git add .gitlab-ci.yml
echo "git commit" echo "git commit"
git commit -m "$commit_msg" commit_message="Auto-generated commit testing"
echo "git commit-tree/reset" commit_message="${commit_message} ${current_branch} (${CI_COMMIT_SHA})"
# Prepare to send the whole working copy. Doing this instead should be faster git commit --message="${commit_message}"
# until we decide to come up with a way of automatically keeping the downstream
# repo in sync with the main one, at which point just pushing a single, new
# commit with the change would be faster.
git reset "$( git commit-tree HEAD^{tree} -m ${commit_msg} )"
echo "git status"
git status
echo "git push" echo "git push"
git push --force "$DOWNSTREAM_CI_REPO" \ git remote add origin "$DOWNSTREAM_CI_REPO"
"___multi_ci___:multi-ci-${current_branch}" git push --force origin "master:multi-ci-${current_branch}"
rm -rf .git
mv "$temp_dir/original-git-dir" .git
git reset --hard HEAD

View file

@ -646,9 +646,8 @@ def release_jobs(parser, args):
'variables': { 'variables': {
'MIRROR_URL': mirror_urls[0], 'MIRROR_URL': mirror_urls[0],
}, },
'image': 'scottwittenburg/spack_ci_generator_alpine',
'script': './bin/rebuild-index.sh', 'script': './bin/rebuild-index.sh',
'tags': ['spack-k8s'] # may want a runner to handle this 'tags': ['spack-post-ci'] # may want a runner to handle this
} }
output_object['rebuild-index'] = final_job output_object['rebuild-index'] = final_job
stage_names.append(final_stage) stage_names.append(final_stage)

View file

@ -67,7 +67,7 @@ def get_s3_session(endpoint_url):
raise SpackError('boto3 module not available') raise SpackError('boto3 module not available')
session = boto3.Session() session = boto3.Session()
s3 = session.resource('s3') s3 = session.resource('s3', endpoint_url=endpoint_url)
bucket_names = [] bucket_names = []
for bucket in s3.buckets.all(): for bucket in s3.buckets.all():