-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
[WFLY-18461] cmt Quickstart Common Enhancements CY2023Q3 #731
Conversation
Hi @marcosgopen. Thanks for your PR. I'm waiting for a wildfly member to verify that this patch is reasonable to test. If it is, they should reply with Once the patch is verified, the new status will be reflected by the I understand the commands that are listed here. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
f0e6e7e
to
3aac8e7
Compare
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.
Besides the pointed changes please also change runtimeName at https://github.com/wildfly/quickstart/pull/731/files#diff-2013e1734bf0fce09cc1ce9a533ea791cc7e3737fcd82b8d8a0d85fc1caff0efL234 , to name. As shown now at https://docs.google.com/document/d/1GFnZrnlwRkwC0HfziRws2UUWR99KdaQayA7Ei12SGN4/edit#heading=h.ob4izb19fxpk
uses: ./.github/workflows/quickstart_ci.yml | ||
with: | ||
QUICKSTART_PATH: cmt | ||
MICROPROFILE: false |
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.
To further enhance the customization and add extensions points there were changes done https://docs.google.com/document/d/1GFnZrnlwRkwC0HfziRws2UUWR99KdaQayA7Ei12SGN4/edit#heading=h.c4iuyi3bm84h
In this case please change this to TEST_PROVISIONED_SERVER: true
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.
Ok, I 've just replaced MICROPROFILE: false with TEST_PROVISIONED_SERVER: true
cmt/pom.xml
Outdated
</configuration> | ||
<executions> | ||
<execution> | ||
<goals> | ||
<goal>integration-test</goal> | ||
<goal>verify</goal> |
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.
please keep the verify goal, it was a bug in the doc
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.
Ok, I've just fixed it.
3aac8e7
to
5fe04b5
Compare
Thanks @emmartins , I addressed your review. |
/ok-to-test |
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.
thanks @marcosgopen, all looks good to me
@kabir FYI this one is ready for OpenShift testing too |
Issue: https://issues.redhat.com/browse/WFLY-18461