Skip to content

Commit

Permalink
Merge pull request #1695 from edx/matthugs/maybe-we-just-didnt-need-e…
Browse files Browse the repository at this point in the history
…dx-secure-for-this-niche-job

fix: remove pull of edx-secure from mongo_mms job
  • Loading branch information
Matt Hughes authored Nov 2, 2023
2 parents b3a7d6b + 6155af0 commit d7a1c91
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 19 deletions.
18 changes: 0 additions & 18 deletions devops/jobs/MongoAgentsUpdate.groovy
Original file line number Diff line number Diff line change
Expand Up @@ -15,10 +15,6 @@ class MongoAgentsUpdate {
'Git repo containing internal overrides')
stringParam('CONFIGURATION_INTERNAL_BRANCH', extraVars.get('CONFIGURATION_INTERNAL_BRANCH', 'master'),
'e.g. tagname or origin/branchname')
stringParam('CONFIGURATION_SECURE_REPO', extraVars.get('CONFIGURATION_SECURE_REPO',"[email protected]:edx-ops/edx-secure.git"),
'Secure Git repo .')
stringParam('CONFIGURATION_SECURE_BRANCH', extraVars.get('CONFIGURATION_SECURE_BRANCH', 'master'),
'e.g. tagname or origin/branchname')
}

wrappers common_wrappers
Expand Down Expand Up @@ -57,20 +53,6 @@ class MongoAgentsUpdate {
relativeTargetDirectory('configuration-internal')
}
}
git {
remote {
url('$CONFIGURATION_SECURE_REPO')
branch('$CONFIGURATION_SECURE_BRANCH')
if (gitCredentialId) {
credentials(gitCredentialId)
}
}
extensions {
cleanAfterCheckout()
pruneBranches()
relativeTargetDirectory('configuration-secure')
}
}
}

steps {
Expand Down
2 changes: 1 addition & 1 deletion devops/resources/mongo-agents-update.sh
Original file line number Diff line number Diff line change
Expand Up @@ -19,4 +19,4 @@ assume-role ${ROLE_ARN}
set -x
cd playbooks

ansible-playbook -i ./ec2.py --limit tag_Name_edx-admin-mms mongo_mms.yml -e@../../configuration-internal/ansible/vars/edx.yml -e@../../configuration-secure/ansible/vars/edx.yml -u ubuntu
ansible-playbook -i ./ec2.py --limit tag_Name_edx-admin-mms mongo_mms.yml -e@../../configuration-internal/ansible/vars/edx.yml ubuntu

0 comments on commit d7a1c91

Please sign in to comment.