Skip to content

Fix pull_licenses_java returning 404 from opensource.org#25515

Merged
chamikaramj merged 1 commit intoapache:masterfrom
bvolpato:licenses-fix
Feb 16, 2023
Merged

Fix pull_licenses_java returning 404 from opensource.org#25515
chamikaramj merged 1 commit intoapache:masterfrom
bvolpato:licenses-fix

Conversation

@bvolpato
Copy link
Copy Markdown
Contributor

@bvolpato bvolpato commented Feb 16, 2023

Apparently opensource.org changed the paths to reach license from https://opensource.org/licenses/mit-license.php -> https://opensource.org/licenses/mit/

This change fixes recent snapshot breakages.

For example: https://ci-beam.apache.org/job/beam_Publish_Beam_SDK_Snapshots/4836/console

11:17:12 ERROR:root:Invalid url for pcollections-2.1.2: http://www.opensource.org/licenses/mit-license.php. Retrying...
11:17:12 Traceback (most recent call last):
11:17:12   File "/home/jenkins/jenkins-slave/workspace/beam_Publish_Beam_SDK_Snapshots/src/sdks/java/container/license_scripts/pull_licenses_java.py", line 58, in pull_from_url
11:17:12     url_read = urlopen(Request(url, headers={

@riteshghorse riteshghorse mentioned this pull request Feb 16, 2023
Merged
3 tasks
@chamikaramj
Copy link
Copy Markdown
Contributor

chamikaramj commented Feb 16, 2023

LGTM. Thanks.

@chamikaramj
Copy link
Copy Markdown
Contributor

chamikaramj commented Feb 16, 2023

Will merge after tests.

@jrmccluskey jrmccluskey mentioned this pull request Feb 16, 2023
Merged
3 tasks
@chamikaramj chamikaramj merged commit 1b1cc44 into apache:master Feb 16, 2023
@bvolpato bvolpato deleted the licenses-fix branch February 16, 2023 18:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@bvolpato @chamikaramj