jakartaee / jaxb-tck

Other
4 stars 16 forks source link

wrong content of the tck zip #36

Closed lukasj closed 3 years ago

lukasj commented 3 years ago

when I unzip the jakarta-xml-binding-tck-3.0.0.zip file, what I see is:

gurunrao commented 3 years ago

@RohitKumarJain - can you look into the content mismatch highlighted by lukas.

RohitKumarJain commented 3 years ago

@gurunrao @lukasj , I am looking into it. Will update with my findings on the same.

RohitKumarJain commented 3 years ago

@gurunrao @lukasj @alwin-joseph , when I am testing the generated bundle directory structure in my local machine it does not contain "?" as part of "XML-binding-tck-3.0.0.zip". Even when I am trying to extract in my Windows machine, again I don't see "?" folder.

While when I am trying to download in my Linux machine (wget https://ci.eclipse.org/jakartaee-tck/job/rjain-jaxb-tck/job/master/6/artifact/bundles/xml-binding-tck-3.0.0.zip) I am observing "?" folder is getting generated.

I tried to debug it further but not sure why "?" folder is generating? Can anyone please help to fix this?

Attaching the screenshots in the documents while investigating this issue. Analysis.docx

lukasj commented 3 years ago

41 should fix remaining issues (including ? directory)

RohitKumarJain commented 3 years ago

@lukasj , I just checked the generated bundle as part of https://ci.eclipse.org/jakartaee-tck/job/jaxb-tck/job/master/63/ and also tried to test the latest bundle from https://download.eclipse.org/ee4j/jakartaee-tck/jakartaee9-eftl/staged-900/jakarta-xml-binding-tck-3.0.0.zip. My finding is below:

lukasj commented 3 years ago

@RohitKumarJain

alwin-joseph commented 3 years ago

@RohitKumarJain

* wrt bundle structure - it now follows other TCK bundles

* wrt `?` folder - can you try to alter your job to only build the tck bundle (no need to wait 8 hours for test runs), archive `jaxb-tck-build/installer-classes.lst` and `jaxb-tck-build/unzip.log` files and modify `ant clean dist_eftl` call in `build_jaxbtck.sh` to `ant -v clean dist_eftl` and rerun your job? that could show us from where the `?` folder is comming...

https://github.com/eclipse-ee4j/jaxb-tck/pull/42 should remove the ? folder. Tested it in a separate job. Awaiting the completion of https://ci.eclipse.org/jakartaee-tck/job/eftl-jaxb-tck-build-run/13/ to refresh the staged bundle.

alwin-joseph commented 3 years ago

@RohitKumarJain

* wrt bundle structure - it now follows other TCK bundles

* wrt `?` folder - can you try to alter your job to only build the tck bundle (no need to wait 8 hours for test runs), archive `jaxb-tck-build/installer-classes.lst` and `jaxb-tck-build/unzip.log` files and modify `ant clean dist_eftl` call in `build_jaxbtck.sh` to `ant -v clean dist_eftl` and rerun your job? that could show us from where the `?` folder is comming...

42 should remove the ? folder. Tested it in a separate job. Awaiting the completion of https://ci.eclipse.org/jakartaee-tck/job/eftl-jaxb-tck-build-run/13/ to refresh the staged bundle.

https://ci.eclipse.org/jakartaee-tck/job/eftl-jaxb-tck-build-run/14 was run instead of above job and https://download.eclipse.org/ee4j/jakartaee-tck/jakartaee9-eftl/staged-900/jakarta-xml-binding-tck-3.0.0.zip was refreshed. Can you please verify if we can promote this bundle. I see no "?" folder.

RohitKumarJain commented 3 years ago

@RohitKumarJain

* wrt bundle structure - it now follows other TCK bundles

* wrt `?` folder - can you try to alter your job to only build the tck bundle (no need to wait 8 hours for test runs), archive `jaxb-tck-build/installer-classes.lst` and `jaxb-tck-build/unzip.log` files and modify `ant clean dist_eftl` call in `build_jaxbtck.sh` to `ant -v clean dist_eftl` and rerun your job? that could show us from where the `?` folder is comming...

42 should remove the ? folder. Tested it in a separate job. Awaiting the completion of https://ci.eclipse.org/jakartaee-tck/job/eftl-jaxb-tck-build-run/13/ to refresh the staged bundle.

https://ci.eclipse.org/jakartaee-tck/job/eftl-jaxb-tck-build-run/14 was run instead of above job and https://download.eclipse.org/ee4j/jakartaee-tck/jakartaee9-eftl/staged-900/jakarta-xml-binding-tck-3.0.0.zip was refreshed. Can you please verify if we can promote this bundle. I see no "?" folder.

@alwin-joseph , I have just verified the structure while taking the .zip from https://download.eclipse.org/ee4j/jakartaee-tck/jakartaee9-eftl/staged-900/jakarta-xml-binding-tck-3.0.0.zip. The structure looks good to me, and thanks for making the fix for "?". According to me, we can promote the bundle. @lukasj @gurunrao , requesting if you also see it once before promoting the bundle.

lukasj commented 3 years ago

yup, all problems are fixed now. Go ahead and promote the TCK binary, please. Thanks!

alwin-joseph commented 3 years ago

yup, all problems are fixed now. Go ahead and promote the TCK binary, please. Thanks!

Thanks. I have promoted the TCK binary to https://download.eclipse.org/ee4j/jakartaee-tck/jakartaee9-eftl/promoted/jakarta-xml-binding-tck-3.0.0.zip .