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

[CI][Release] Source Release and Merge Script jobs are currently failing on GHA #14949

Closed
raulcd opened this issue Dec 14, 2022 · 7 comments · Fixed by #14957
Closed

[CI][Release] Source Release and Merge Script jobs are currently failing on GHA #14949

raulcd opened this issue Dec 14, 2022 · 7 comments · Fixed by #14957

Comments

@raulcd
Copy link
Member

raulcd commented Dec 14, 2022

Describe the bug, including details regarding any error messages, version, and platform.

This has been raised on Zulip.
The jobs for Source Release and Merge Script are failing on several PRs and master. Example of failures:

From what I understand the error seems to be around the following not being found (but I might be mistaken as the output is quite difficult to follow):

 -   :path=>"java/adapter/avro/pom.xml"},
-  {:hunks=>
-    [["-        <version>11.0.0-SNAPSHOT</version>",
-      "+        <version>12.0.0-SNAPSHOT</version>"]],
-   :path=>"java/adapter/jdbc/pom.xml"},
-  {:hunks=>
-    [["-        <version>11.0.0-SNAPSHOT</version>",
-      "+        <version>12.0.0-SNAPSHOT</version>"]],
-   :path=>"java/adapter/orc/pom.xml"},
-  {:hunks=>
-    [["-    <version>11.0.0-SNAPSHOT</version>",
-      "+    <version>12.0.0-SNAPSHOT</version>"]],

I can't find any change lately that might be the cause. The first commit on master that seems to fail due to this issue is unrelated: 4647739

Component(s)

Continuous Integration

@raulcd
Copy link
Member Author

raulcd commented Dec 14, 2022

I can't seem to reproduce locally:

(pyarrow-dev) ~/code/arrow  (master)  $ git pull upstream master
From github.com:apache/arrow
 * branch            master     -> FETCH_HEAD
Already up to date.
(pyarrow-dev) ~/code/arrow  (master)  $ ci/scripts/release_test.sh $(pwd)
+ arrow_dir=/home/raulcd/code/arrow
+ pushd /home/raulcd/code/arrow
~/code/arrow ~/code/arrow
+ dev/release/run-test.rb
Loaded suite release
Started
.........
Finished in 26.797402892 seconds.
---------------------------------------------------------------------------------------------------------------------------------------------------------------
9 tests, 9 assertions, 0 failures, 0 errors, 0 pendings, 0 omissions, 0 notifications
100% passed
---------------------------------------------------------------------------------------------------------------------------------------------------------------
0.34 tests/s, 0.34 assertions/s
+ popd
~/code/arrow

@raulcd
Copy link
Member Author

raulcd commented Dec 14, 2022

I've checked and the GitHub runner seems to be exactly the same (no version updates) between a successful run:
https://github.com/apache/arrow/actions/runs/3692289538/jobs/6251037794
and a failure one:
https://github.com/apache/arrow/actions/runs/3693047779/jobs/6252584126

@raulcd
Copy link
Member Author

raulcd commented Dec 14, 2022

@kou any idea what could be the issue?

@kou
Copy link
Member

kou commented Dec 14, 2022

It seems that mvn version:set in dev/release/utils-prepare.sh doesn't update java/adapter/*/pom.xml but why...?

@kou
Copy link
Member

kou commented Dec 14, 2022

I reran the succeeded build and it failed now: https://github.com/apache/arrow/actions/runs/3692289538/jobs/6264959209

It means that this is not related to our changes. It's related to CI environment.

@kou
Copy link
Member

kou commented Dec 15, 2022

I think that mojohaus/versions#848 is the problem.

@raulcd
Copy link
Member Author

raulcd commented Dec 15, 2022

This seems to work now on my last PR the jobs were successful: https://github.com/apache/arrow/actions/runs/3706180881/jobs/6281060893

kou added a commit that referenced this issue Dec 15, 2022
This doesn't solve GH-14949 (CI failure) because it's a problem of  MojoHaus Versions Maven Plugin: mojohaus/versions#848

This adds more debug information on test failure.
* Closes: #14949

Authored-by: Sutou Kouhei <kou@clear-code.com>
Signed-off-by: Sutou Kouhei <kou@clear-code.com>
@kou kou added this to the 11.0.0 milestone Dec 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants