Skip to content

Commit

Permalink
Merge branch 'ko3n1g/ci/fix-notify-image' into 'main'
Browse files Browse the repository at this point in the history
ci: Use `curl-jq` for notify step

See merge request ADLR/megatron-lm!2387
  • Loading branch information
ko3n1g committed Nov 25, 2024
2 parents 47806ab + 2f2b1f1 commit e21ce31
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 4 deletions.
2 changes: 1 addition & 1 deletion .gitlab/stages/01.test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -203,7 +203,7 @@ test:pyt(DEV)_mcore(0.9.0):

test:notify_unit_tests:
extends: [.test_rules]
image: ${UTILITY_IMAGE}:${CI_PIPELINE_ID}
image: badouralix/curl-jq
needs:
- test:pyt(LTS)_mcore(latest)
- test:pyt(DEV)_mcore(latest)
Expand Down
4 changes: 1 addition & 3 deletions .gitlab/stages/02.functional-tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -122,14 +122,12 @@ functional:run_dev:

.notify:
extends: [.functional_tests_rules]
image: ${UTILITY_IMAGE}:${CI_PIPELINE_ID}
image: badouralix/curl-jq
needs:
- functional:run_lts
- functional:run_dev
tags:
- mcore-docker-node-small
before_script:
- jet secrets jwt-login jwt/nvidia/gitlab-master adlr-megatron-lm-ci $VAULT_JWT_TOKEN
variables:
WEBHOOK_URL: ${MCORE_NOTIFICATION_HOOK}
RO_API_TOKEN: ${PROJECT_ACCESS_TOKEN_MCORE}
Expand Down

0 comments on commit e21ce31

Please sign in to comment.