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
CLOUD-3091 - Update builds to not use cacher #58
Conversation
@@ -7,7 +7,7 @@ execute: | |||
user: '185' | |||
|
|||
artifacts: | |||
- path: txn-recovery-marker-jdbc-common-1.1.2.Final-redhat-00001.jar | |||
- name: txn-recovery-marker-jdbc-common-1.1.2.Final-redhat-00001.jar |
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.
I suggest to use something like this:
- name: txn-recovery-marker-jdbc-common
target: txn-recovery-marker-jdbc-common-1.1.2.Final-redhat-00001.jar
md5: cd68ad886a759d21f8dd0cb7646601ee
Same applies to other artifacts.
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.
Ah, I see, so it can be overriden via name. Clever :)
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.
Exactly!
I'm dismissing my review not because I think this is wrong but because reading the comments on the JIRA I realize I don't know if there are problems.
@bstansberry this PR should be all set I think, not sure what the other issues might be from reading the jira. |
FYI, this is causing issues for folks building images, someone approve it please :) |
* CLOUD-3091 - Update builds to not use cacher * update use name / target / md5
CLOUD-3175 - Update builds to not use cacher (#58) [7.2.x]
Adding tests for GC_METASPACE_SIZE
https://issues.jboss.org/browse/CLOUD-3091
Signed-off-by: Ken Wills kwills@redhat.com