-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
CURATOR-634: Re-introduce TravisCI for testing on Linux ARM64 #412
Conversation
Signed-off-by: Martin Tzvetanov Grigorov <mgrigorov@apache.org>
This way it won't be checked by maven-rat-plugin Signed-off-by: Martin Tzvetanov Grigorov <mgrigorov@apache.org>
4dbbb6b
to
922d524
Compare
Any feedback on the proposal to (re-)use TravisCI for testing on Linux ARM64 ? |
I'm unfamiliar with how the old Travis CI set up. Thus cc @Randgalt @eolivelli please take a look. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@martin-g I learn more about running travis tasks on Apache projects recently and agree that this patch is good to merge. Comments inline.
arch: arm64-graviton2 | ||
group: edge | ||
virt: vm | ||
language: generic |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
language: generic | |
language: java |
Why not use a java env? In this approach you don't have to install maven manually.
Because this way it will use a rather old version of Maven. I don't
remember now whether Curator uses maven-enforcer-plugin or not.
…On Tue, Jul 5, 2022, 18:21 tison ***@***.***> wrote:
***@***.**** commented on this pull request.
@martin-g <https://github.com/martin-g> I learn more about running travis
tasks on Apache projects recently and agree that this patch is good to
merge. Comments inline.
------------------------------
In .travis.yml
<#412 (comment)>:
> +#
+# http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing,
+# software distributed under the License is distributed on an
+# "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+# KIND, either express or implied. See the License for the
+# specific language governing permissions and limitations
+# under the License.
+
+os: linux
+dist: focal
+arch: arm64-graviton2
+group: edge
+virt: vm
+language: generic
⬇️ Suggested change
-language: generic
+language: java
Why not use a java env? In this approach you don't have to install maven
manually.
—
Reply to this email directly, view it on GitHub
<#412 (review)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AABYUQRMS5NC4OQDG4VABFTVSRHHRANCNFSM5PQQJA4Q>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
@martin-g Perhaps you can introduce a maven wrapper in curator repository first. It will be a good solution for such issues. |
I'd rather just close the PR than play that limbo. |
No description provided.