Skip to content

push to dockerhub#8322

Merged
AndWeHaveAPlan merged 8 commits intomasterfrom
es/parity-pr-images
Apr 25, 2025
Merged

push to dockerhub#8322
AndWeHaveAPlan merged 8 commits intomasterfrom
es/parity-pr-images

Conversation

@AndWeHaveAPlan
Copy link
Copy Markdown
Contributor

@AndWeHaveAPlan AndWeHaveAPlan commented Apr 24, 2025

Push images to dockerhub paritypr org in addition to private google registry
fix https://github.com/paritytech/devops/issues/3976

@AndWeHaveAPlan AndWeHaveAPlan added R0-no-crate-publish-required The change does not require any crates to be re-published. A5-run-CI Run CI on draft PR labels Apr 24, 2025
@AndWeHaveAPlan
Copy link
Copy Markdown
Contributor Author

dockerhub, google registry (for 299e45c)

@AndWeHaveAPlan AndWeHaveAPlan marked this pull request as ready for review April 24, 2025 14:39
@AndWeHaveAPlan AndWeHaveAPlan requested review from a team as code owners April 24, 2025 14:39
image-name: "europe-docker.pkg.dev/parity-ci-2024/temp-images/test-parachain"
image-name: "test-parachain"
dockerfile: "docker/dockerfiles/test-parachain_injected.Dockerfile"
username: ${{ secrets.PARITYPR_DOCKERHUB_USERNAME }}
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

It's going to fail for PRs from forks. I'd add a separate job that'll push images for master only.

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

we also need optional image push from PR branches not only master.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

updated, now login/push only if secrets are present

@AndWeHaveAPlan AndWeHaveAPlan requested a review from alvicsam April 24, 2025 15:32
@AndWeHaveAPlan AndWeHaveAPlan added this pull request to the merge queue Apr 25, 2025
Merged via the queue into master with commit 0171ff8 Apr 25, 2025
241 of 248 checks passed
@AndWeHaveAPlan AndWeHaveAPlan deleted the es/parity-pr-images branch April 25, 2025 16:08
wassimans pushed a commit to wassimans/polkadot-sdk that referenced this pull request Apr 27, 2025
Push images to dockerhub paritypr org in addition to private google
registry
fix https://github.com/paritytech/devops/issues/3976
ordian added a commit that referenced this pull request Apr 28, 2025
* master: (120 commits)
  [CI] Improve GH build status checking (#8331)
  [CI/CD] Use original PR name in prdoc check for the backport PR's to the stable branches (#8329)
  Add new host APIs set_storage_or_clear and get_storage_or_zero (#7857)
  push to dockerhub (#8322)
  Snowbridge - V1 - Adds 2 hop transfer to Rococo (#7956)
  [AHM] Prepare `election-provider-multi-block` for full lazy data deletion (#8304)
  Check umbrella version (#8250)
  [AHM] Fully bound staking async (#8303)
  migrate parachain-templates tests to `gha` (#8226)
  staking-async: add missing new_session_genesis (#8310)
  New NFT traits: granular and abstract interface (#5620)
  Extract create_pool_with_native_on macro to common crate (#8289)
  XCMP: use batching when enqueuing inbound messages (#8021)
  Snowbridge - Tests refactor (#8014)
  Allow configuration of worst case buy execution weight (#7944)
  Fix faulty pre-upgrade migration check in pallet-session (#8294)
  [pallet-revive] add get_storage_var_key for variable-sized keys (#8274)
  add poke_deposit extrinsic to pallet-recovery (#7882)
  `txpool`: use tracing for structured logging (#8001)
  [revive] eth-rpc refactoring (#8148)
  ...
castillax pushed a commit that referenced this pull request May 12, 2025
Push images to dockerhub paritypr org in addition to private google
registry
fix paritytech/devops#3976
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A5-run-CI Run CI on draft PR R0-no-crate-publish-required The change does not require any crates to be re-published.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants