Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

KAFKA-17735: release.py must not use home.apache.org #17421

Merged
merged 1 commit into from
Oct 8, 2024

Conversation

cmccabe
Copy link
Contributor

@cmccabe cmccabe commented Oct 8, 2024

Previously, Apache Kafka was uploading release candidate (RC) artifacts
to users' home directories on home.apache.org. However, since this
resource has been decommissioned, we need to follow the standard
approach of putting release candidate artifacts into the appropriate
subversion directory, at https://dist.apache.org/repos/dist/dev/kafka/.
(This is distinct from the release directories at
https://dist.apache.org/repos/dist/release/kafka/)

Previously, Apache Kafka was uploading release candidate (RC) artifacts
to users' home directories on home.apache.org. However, since this
resource has been decommissioned, we need to follow the standard
approach of putting release candidate artifacts into the appropriate
subversion directory, at https://dist.apache.org/repos/dist/dev/kafka/.
(This is distinct from the release directories at
https://dist.apache.org/repos/dist/release/kafka/)
@@ -70,10 +70,12 @@
import gpg
import notes
import preferences
import sftp
Copy link
Member

Choose a reason for hiding this comment

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

did we need to change from sftp to svn for this or is it just an additional change?

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 do not have any place to sftp to any more, now that home.apache.org is gone.

@jolshan
Copy link
Member

jolshan commented Oct 8, 2024

Have we run the new code to confirm this uploads correctly?

@cmccabe
Copy link
Contributor Author

cmccabe commented Oct 8, 2024

Have we run the new code to confirm this uploads correctly?

yes. I have tested this. since it's uploading to the devel section of svn a test run is no big deal.

@jolshan
Copy link
Member

jolshan commented Oct 8, 2024

As an aside, we can probably mark https://issues.apache.org/jira/browse/KAFKA-17328 as a duplicate.

@cmccabe cmccabe merged commit 8af063a into apache:3.9 Oct 8, 2024
1 check was pending
@cmccabe cmccabe deleted the KAFKA-17735 branch October 8, 2024 22:41
cmccabe added a commit that referenced this pull request Oct 8, 2024
Previously, Apache Kafka was uploading release candidate (RC) artifacts
to users' home directories on home.apache.org. However, since this
resource has been decommissioned, we need to follow the standard
approach of putting release candidate artifacts into the appropriate
subversion directory, at https://dist.apache.org/repos/dist/dev/kafka/.

Reviewers: Justine Olshan <[email protected]>
jlprat pushed a commit that referenced this pull request Oct 10, 2024
Previously, Apache Kafka was uploading release candidate (RC) artifacts
to users' home directories on home.apache.org. However, since this
resource has been decommissioned, we need to follow the standard
approach of putting release candidate artifacts into the appropriate
subversion directory, at https://dist.apache.org/repos/dist/dev/kafka/.

Reviewers: Justine Olshan <[email protected]>
tedyu pushed a commit to tedyu/kafka that referenced this pull request Jan 6, 2025
Previously, Apache Kafka was uploading release candidate (RC) artifacts
to users' home directories on home.apache.org. However, since this
resource has been decommissioned, we need to follow the standard
approach of putting release candidate artifacts into the appropriate
subversion directory, at https://dist.apache.org/repos/dist/dev/kafka/.

Reviewers: Justine Olshan <[email protected]>
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