Skip to content

fix: inject dockerhub creds#12072

Merged
ludamad merged 5 commits intomasterfrom
ad/dockerhub-inject
Feb 18, 2025
Merged

fix: inject dockerhub creds#12072
ludamad merged 5 commits intomasterfrom
ad/dockerhub-inject

Conversation

@ludamad
Copy link
Collaborator

@ludamad ludamad commented Feb 18, 2025

No description provided.

@ludamad ludamad requested a review from charlielye as a code owner February 18, 2025 13:25
@ludamad ludamad enabled auto-merge (squash) February 18, 2025 13:25
source ci3/source

if [ -n "${DOCKERHUB_PASSWORD:-}" ] && [ "$REF_NAME" == "master" ] ; then
echo ${{ secrets.DOCKERHUB_PASSWORD }} | docker login -u aztecprotocolci --password-stdin
Copy link
Collaborator Author

@ludamad ludamad Feb 18, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we restrict to master or release tags to give less surface area from external PRs

@ludamad ludamad disabled auto-merge February 18, 2025 13:34
@ludamad ludamad merged commit 31e1eda into master Feb 18, 2025
8 of 10 checks passed
@ludamad ludamad deleted the ad/dockerhub-inject branch February 18, 2025 13:34
TomAFrench added a commit that referenced this pull request Feb 19, 2025
* master: (245 commits)
  chore: Fix unbound CI variable on release image bootstrap (#12095)
  fix: dry run on grind (#12088)
  fix(spartan): eth-execution logging (#12094)
  fix: aws_handle_evict recovery & termination (#12086)
  chore: Use native acvm when available on orchestrator tests (#11560)
  refactor: function macros cleanup (#12066)
  refactor: remove `addNullifiedNote` from pxe (#11822)
  fix: `#[aztec]` macro warnings (#12038)
  refactor!: Notes implementing `Packable<N>` (#12004)
  chore(ops): add gcloud cli into devbox base image (#12082)
  fix: hotfix grinding
  fix: L1 deployment on reth (#12060)
  fix: Add missing bootstrap fast aliases (#12078)
  fix: hash_str caching (#12074)
  fix: Naive attempt to fix nightly deployments (#12079)
  fix: kind smoke (#12084)
  refactor!: nuking `NoteHeader` (#11942)
  fix: inject dockerhub creds (#12072)
  feat(docs): Note discovery concepts page (#11760)
  chore: cleanup libp2p logger (#12058)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants