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

Detect overall PCT failures #277

Merged
merged 6 commits into from
Jul 31, 2020
Merged

Detect overall PCT failures #277

merged 6 commits into from
Jul 31, 2020

Conversation

jglick
Copy link
Member

@jglick jglick commented Jul 20, 2020

When PCT compilation succeeded but surefire:test or one of its preparatory goals crashed (i.e., not just some test failures), we were silently skipping that plugin, and it seems there were several of these. Noticed in #179. At root this seems like a flaw in PCT which I am just working around here.

@timja
Copy link
Member

timja commented Jul 20, 2020

My browser just chokes trying to look at the log, and blueocean keeps jumping to the bottom -.-

@raul-arabaolaza
Copy link

@jglick LGTM, I can confirm that is an issue that we have solved in other places in similar ways

@jetersen
Copy link
Member

@jglick I'd like to understand how to get to those partial logs.

This was referenced Jul 27, 2020
@jglick
Copy link
Member Author

jglick commented Jul 27, 2020

how to get to those partial logs

We could archiveArtifacts them I suppose, or try to inline them into a CDATA section in the synthetic test report, or you can just look at the build log as in #277 (comment).

@jetersen
Copy link
Member

@jglick I was more referring on how to navigate to them in the Jenkins UI

@jglick
Copy link
Member Author

jglick commented Jul 27, 2020

Go to Pipeline Steps, search for the branch corresponding to the test failure, look for the … && bash pct.sh step, and click the terminal icon next to it.

@bitwiseman
Copy link
Contributor

@jglick @oleg-nenashev

So, now we have three failures. Looking at https://ci.jenkins.io/job/Tools/job/bom/job/PR-277/3/ here are the logs pulled out:

I'm not even sure how to read the failures. When this kind of failure occurs do we want/need to rerun with mvn -X ?

@timja
Copy link
Member

timja commented Jul 30, 2020

the ssh-credentials one looks like a straightforward one to do with the jcasc test harness

@jglick
Copy link
Member Author

jglick commented Jul 30, 2020

do we want/need to rerun with mvn -X ?

You can run PCT for one LTS line and plugin locally quite easily: https://github.com/jenkinsci/bom/#pct

The matrix-project one looks simple enough: just needs to be updated to a new parent POM to pick up jenkinsci/jenkins-test-harness#79.

@jglick
Copy link
Member Author

jglick commented Jul 30, 2020

…which is blocked in #274 on some weirdness I have not dealt with yet: #251 (comment)

@jglick
Copy link
Member Author

jglick commented Jul 30, 2020

The ssh-credentials issue may be because in #110 we stopped updating this plugin. Running git bisect now…

@timja
Copy link
Member

timja commented Jul 30, 2020

@jglick #287

@jglick
Copy link
Member Author

jglick commented Jul 30, 2020

Flakes?

@jglick jglick marked this pull request as ready for review July 30, 2020 22:11
@timja timja merged commit 4c1c157 into jenkinsci:master Jul 31, 2020
@jglick jglick deleted the PCT-overall branch July 31, 2020 14:03
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.

5 participants