Skip to content

Conversation

@breskeby
Copy link
Contributor

@breskeby breskeby commented Apr 10, 2025

This adds a specific docker context that we use for docker hub. For now this only removes specific docker labels that should not be used in docker hub.

@breskeby breskeby requested a review from a team as a code owner April 10, 2025 16:27
@breskeby breskeby added >non-issue :Delivery/Build Build or test infrastructure Team:Delivery Meta label for Delivery team auto-backport Automatically create backport pull requests when merged v8.19.0 v9.1.0 labels Apr 10, 2025
@breskeby breskeby self-assigned this Apr 10, 2025
@breskeby breskeby added v9.0.0 and removed v8.19.0 labels Apr 10, 2025
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-delivery (Team:Delivery)

@breskeby breskeby requested a review from jozala April 10, 2025 16:29
$VERSION_QUALIFIER_ARG \
buildReleaseArtifacts \
exportCompressedDockerImages \
exportDockerContexts \
Copy link
Contributor Author

Choose a reason for hiding this comment

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

we implicitly build those contexts as part of building the images. Since they are deliverables, we want to build them explicitly. Also, we now build the docker hub specific context but not a corresponding image for that

Copy link
Contributor

@jozala jozala left a comment

Choose a reason for hiding this comment

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

Just a minor typo found, but I believe it is worth fixing.

@breskeby breskeby requested a review from jozala April 11, 2025 08:00
@breskeby breskeby merged commit 31995ac into elastic:main Apr 11, 2025
17 checks passed
@elasticsearchmachine
Copy link
Collaborator

💔 Backport failed

Status Branch Result
9.0 Commit could not be cherrypicked due to conflicts

You can use sqren/backport to manually backport by running backport --upstream elastic/elasticsearch --pr 126625

breskeby added a commit to breskeby/elasticsearch that referenced this pull request Apr 11, 2025
The labels we require for openshift are incompatible with docker hub policies. therefore we need to use different docker contexts for those.

(cherry picked from commit 31995ac)
@breskeby
Copy link
Contributor Author

💚 All backports created successfully

Status Branch Result
9.0

Questions ?

Please refer to the Backport tool documentation

elasticsearchmachine pushed a commit that referenced this pull request Apr 11, 2025
The labels we require for openshift are incompatible with docker hub policies. therefore we need to use different docker contexts for those.

(cherry picked from commit 31995ac)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-backport Automatically create backport pull requests when merged backport pending :Delivery/Build Build or test infrastructure >non-issue Team:Delivery Meta label for Delivery team v9.0.0 v9.1.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants