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

fix(ci): add ITEST_IMG_VERSION arg to repeated-tests bash script #1153

Merged
merged 3 commits into from
Oct 24, 2022

Conversation

maxcao13
Copy link
Member

Fixes #1152

Also, I didn't mention this before but there is a mismatch of the itest output filename logs between the bash script and the mvn build executions.

That's why in the split-container-pr #1138, I had to change the run: ls -1dt target/cryostat-itest-*.log | head -n1 | xargs cat -> cryostat-itests.

The pom.xml here on this line:

https://github.com/cryostatio/cryostat/blob/132459fbefed742f00c26058b184b2889f790aa3/pom.xml#L633

outputs the file as cryostat-itest-timestamp.log while the script does cryostat-itests-timestamp.log since cryostat.itest.containerName="cryostat-itest".

Which should be changed here?

@andrewazores
Copy link
Member

I don't think it matters much either way. I suppose the pom.xml is the more "correct" one and the script and CI configs should be adjusted to match.

@andrewazores andrewazores merged commit 911e01d into cryostatio:main Oct 24, 2022
@maxcao13 maxcao13 deleted the pull-from-local-ci branch October 24, 2022 23:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Task] CI pulls from quay.io instead of from local podman repository
2 participants