Skip to content

[SPARK-49813] Publish Apache Spark 3.5.3 to docker registry - #72

Closed
itholic wants to merge 4 commits into
apache:masterfrom
itholic:docker_for_3.5.3
Closed

[SPARK-49813] Publish Apache Spark 3.5.3 to docker registry#72
itholic wants to merge 4 commits into
apache:masterfrom
itholic:docker_for_3.5.3

Conversation

@itholic

Copy link
Copy Markdown
Contributor

What changes were proposed in this pull request?

This PR proposes to publish Apache Spark 3.5.3 to docker registry

Why are the changes needed?

This is one of required release step for Apache Spark

Does this PR introduce any user-facing change?

No

How was this patch tested?

CI

@itholic

Copy link
Copy Markdown
ContributorAuthor

cc @dongjoon-hyun would you mind taking a look when you find some time?

@HyukjinKwon

Copy link
Copy Markdown
Member

cc @Yikun

Comment thread3.5.3/scala2.12-java11-ubuntu/Dockerfile Outdated
@itholic

Copy link
Copy Markdown
ContributorAuthor

@Yikun would mind checking CI why it's still failing after rerunning ./add-dockerfiles.sh 3.5.3 ??

@Yikun

Yikun commented Sep 26, 2024

Copy link
Copy Markdown
Member

GPG validate Error, you can see #63 (comment) as reference. After this resolved, you could also add a FAQ to README.

@itholic

Copy link
Copy Markdown
ContributorAuthor

Thanks @Yikun ! Just uploaded the public key

@itholic

Copy link
Copy Markdown
ContributorAuthor

CI passed. Now can we just merge it?

@dongjoon-hyundongjoon-hyun left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

+1, LGTM.

@YikunYikun left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

It would be better to add a jira number in PR title, otherwise LGTM

@itholicitholic changed the title Publish Apache Spark 3.5.3 to docker registry[SPARK-49813] Publish Apache Spark 3.5.3 to docker registrySep 27, 2024
@itholic

Copy link
Copy Markdown
ContributorAuthor

Thanks! Merged to master

@dongjoon-hyun

dongjoon-hyun commented Sep 27, 2024

Copy link
Copy Markdown
Member

For the record, I published the images to apache/spark, @itholic .

@itholic

Copy link
Copy Markdown
ContributorAuthor

Got it. Thanks, @dongjoon-hyun !

ravisanwal added a commit to ravisanwal/official-images that referenced this pull request Oct 8, 2024
apache/spark PR for 3.5.3 is here apache/spark-docker#72
Sign up for freeto 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.

4 participants

@itholic@HyukjinKwon@Yikun@dongjoon-hyun