Skip to content

Commit

Permalink
[ci][docker] Fix deploy to tlcpackstaging on Docker Hub
Browse files Browse the repository at this point in the history
  • Loading branch information
driazati committed Aug 2, 2022
1 parent b8893b5 commit ef76ff0
Show file tree
Hide file tree
Showing 4 changed files with 15 additions and 11 deletions.
14 changes: 8 additions & 6 deletions Jenkinsfile

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

4 changes: 2 additions & 2 deletions ci/jenkins/Deploy.groovy.j2
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@ stage('Build packages') {

def update_docker(ecr_image, hub_image) {
if (!ecr_image.contains("amazonaws.com")) {
sh("echo Skipping '${ecr_image}' since it doesn't look like an ECR image")
sh("echo \"Skipping '${ecr_image}' -> '${hub_image}' since it doesn\'t look like an ECR image\"")
return
}
docker_init(ecr_image)
Expand Down Expand Up @@ -86,7 +86,7 @@ def deploy() {
}
}
}
if (env.BRANCH_NAME == 'main' && env.DEPLOY_DOCKER_IMAGES == 'yes' && rebuild_docker_images && upstream_revision != null) {
if (env.BRANCH_NAME == 'PR-12282' && env.DEPLOY_DOCKER_IMAGES == 'yes' && rebuild_docker_images && upstream_revision != null) {
node('CPU') {
ws({{ m.per_exec_ws('tvm/deploy-docker') }}) {
try {
Expand Down
6 changes: 3 additions & 3 deletions ci/jenkins/Jenkinsfile.j2
Original file line number Diff line number Diff line change
Expand Up @@ -114,10 +114,10 @@ if (rebuild_docker_images) {
build_docker_images()
}

lint()
// lint()

build()
// build()

test()
// test()

deploy()
2 changes: 2 additions & 0 deletions ci/jenkins/Prepare.groovy.j2
Original file line number Diff line number Diff line change
Expand Up @@ -180,6 +180,8 @@ def prepare() {
script: './tests/scripts/git_change_docker.sh',
label: 'Check for any docker changes',
)
rebuild_docker_images = true;

if (skip_ci) {
// Don't rebuild when skipping CI
rebuild_docker_images = false
Expand Down

0 comments on commit ef76ff0

Please sign in to comment.