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

Nightly build files not publishing? #64

Closed
jasonk000 opened this issue Aug 15, 2024 · 5 comments
Closed

Nightly build files not publishing? #64

jasonk000 opened this issue Aug 15, 2024 · 5 comments

Comments

@jasonk000
Copy link
Contributor

I notice the nightly build doesn't seem to be updating on the MAT update website?

https://download.eclipse.org/mat/snapshots/update-site/

Last publish date looks to be June 6.

  ..      
  binary 11.4K 2024-06-06 11:28  
  features 72K 2024-06-06 11:28  
  plugins 15.1M 2024-06-06 11:28  
  artifacts.jar 5.6k 2024-06-06 11:28  
  artifacts.xml.xz 4.8k 2024-06-06 11:28  
  content.jar 24.5k 2024-06-06 11:27  
  content.xml.xz 10.9k 2024-06-06 11:28  
  p2.index 172 2024-06-06 11:28

Do we need to finalise the work in #55 GH Actions workflow? I had assumed the old pipelines would continue to run post-migration.

@jasonk000
Copy link
Contributor Author

jasonk000 commented Aug 31, 2024

@krumts it looks like this is dash/cyclone issue, are you able to suggest any things to investigate?

Caused by: java.lang.NullPointerException: Cannot invoke "java.util.Map.keySet()" because "this.approvedLicenses" is null

The keySet() call has been throughout the code for many years, so this seems to be a new issue.

I can see you made this commit -- 4535fc2, but I do not see exactly where we specify the license. I assume since we do not provide one that it resorts to the dash-license-plugin source, which looks to be here

public static final String DEFAULT_APPROVED_LICENSES_URL = "https://www.eclipse.org/legal/licenses.json";

I tried to set it directly, but no success.

                        <plugin>
                                <groupId>org.eclipse.dash</groupId>
                                <artifactId>license-tool-plugin</artifactId>
                                <version>1.0.2</version>
                                <configuration>
                                        <licenses>https://www.eclipse.org/legal/licenses.json</licenses>
                                </configuration>
                                ...
                        </plugin>

Only thing I see suspicious in the logs references j9 and SWT.

[WARNING] Problems resolving provisioning plan.: [Unable to satisfy dependency from org.eclipse.swt 3.121.0.v20220829-1402 to org.eclipse.equinox.p2.iu; org.eclipse.swt.gtk.linux.ppc64le [3.121.0.v20220829-1402,3.121.0.v20220829-1402].]
[WARNING] Problems resolving provisioning plan.: [Unable to satisfy dependency from org.eclipse.swt 3.121.0.v20220829-1402 to org.eclipse.equinox.p2.iu; org.eclipse.swt.gtk.linux.aarch64 [3.121.0.v20220829-1402,3.121.0.v20220829-1402].]
[WARNING] Bundle-ClassPath entry lib/j9ddr-structs.jar does not exist in /Users/jkoch/.m2/repository/p2/osgi/bundle/com.ibm.dtfj.j9/1.12.29003.202006111057/com.ibm.dtfj.j9-1.12.29003.202006111057.jar
[WARNING] Bundle-ClassPath entry lib/j9ddr-structs.jar does not exist in /Users/jkoch/.m2/repository/p2/osgi/bundle/com.ibm.dtfj.j9/1.12.29003.202006111057/com.ibm.dtfj.j9-1.12.29003.202006111057.jar

I can see the history here:
https://bugs.eclipse.org/bugs/show_bug.cgi?id=582480

@krumts
Copy link
Contributor

krumts commented Sep 10, 2024

@jasonk000 , sorry for the delayed reply. Something changed and both local and jenkins builds are failing reproducibly - also for older MAT commits, which used to build fine.
I figured out, that the default url for licenses which you pasted replies with a 302 redirect to https://www.eclipse.org/legal/licenses/licenses.json (i.e. one folder "licenses" deeper). Is looks like the dash plugin doesn't handle the redirects correct.
Adding the new URL explicitly (as you did with the old one) fixed my local builds.
I'll push a fix for it and hope this would fix also the jenkins build.
I'll see if the issue was reported on dash and do so if this is not yet the case.

krumts added a commit that referenced this issue Sep 10, 2024
Explicitly provide the approved licenses URL, as the dash plugin failed
to follow a redirect and failed the build.

Signed-off-by: Krum Tsvetkov <[email protected]>
@krumts krumts closed this as completed in 49089e1 Sep 10, 2024
@krumts
Copy link
Contributor

krumts commented Sep 10, 2024

#66 fixed the build and I confirmed that also the contents of the update site got updated.

krumts added a commit that referenced this issue Sep 10, 2024
The 1.1.0 version of the licence-tool-plugin works fine with the
redirected approved licenses default URL

Signed-off-by: Krum Tsvetkov <[email protected]>
krumts added a commit that referenced this issue Sep 10, 2024
The 1.1.0 version of the licence-tool-plugin works fine with the
redirected approved licenses default URL

Signed-off-by: Krum Tsvetkov <[email protected]>
@jasonk000
Copy link
Contributor Author

Ah, great work, I'd never checked for redirects, and must have mixed up my report. Thank you for taking care of it.

@krumts
Copy link
Contributor

krumts commented Sep 10, 2024

It turned out, that the latest version of the license-tool-plugin works fine with the redirects. I just didn't see that this version exists (although I checked :) ). So, I made a second change - upgrading to the latest version and removed the explicit URL config.

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

No branches or pull requests

2 participants