parallelio: new version#35553
parallelio: new version#35553tldahlgren merged 3 commits intospack:developfrom theurich:parallelio_2_5_10
Conversation
|
@jedwards4b - just curious, since I am very new to Spack... when I opened this PR, did you get notified automatically since you are mentioned as maintainer in the parallelio package? Or how does this mechanism work? I figured this simple change was a good way for me to learn the Spack work-flow of how to contribute. Thanks! |
jedwards4b
left a comment
There was a problem hiding this comment.
I think that you need to add me as a reviewer to notify me.
|
Somehow I could not add any reviewers. This is strange. Now I just requested a re-review from you, because I had to rebase my branch to the current develop. Nothing else changed. |
|
merge will be blocked until one of the spack reviewers looks at it. |
|
Okay maybe what I should have done is mention you in the PR when I first opened it. Again, I don't think I was able to add you as a reviewer, but I could have mentioned you. Then you can review and this kicks things off on the Spack side? I sayu this b/c I see spack-bot app active after we had some action on the PR. .... still learning ... |
|
@spackbot run pipeline |
|
I'm sorry, gitlab does not have your latest revision yet, I can't run that pipeline for you right now. One likely possibility is that your PR pipeline has been temporarily deferred, in which case, it is awaiting a Please check the gitlab commit status message to see if more information is available. DetailsUnexpected response from gitlab: {'message': '404 Commit Not Found'} |
|
@tldahlgren - I see it says that the branch is out-of-date with the base branch. Should I manually hit the "Update branch" button, or is this going to happen automatically? Thank you for your guidance. |
Either is fine. I still get an error trying to look at the pipeline results so an update may help. |
|
Okay, updated the branch against develop. Let's hope it helps to move it forward. |
🤞 |
|
@jedwards4b @tldahlgren @alalazo - is it normal for a small change in a package to take this long to be merged? I am just curious because this is my first contribution to Spack, and I am wondering if I am missing something on my end I should be doing to move it forward. But if this is normal, then all good. Thanks! |
|
It's normal - consider that the repo currently has over 600 open PRs. |
|
@theurich You have also been unlucky, in that we are currently resolving an issue with our gitlab builders that prevented many PRs to pass CI in the last 2-3 days. |
|
@spackbot run pipeline |
|
I've started that pipeline for you! |
|
Thanks everyone... I am learning. |
You're welcome.
Yes, you should be able to use that spackbot command. BTW. I'm told re-running the pipeline isn't going to work until those working on the gitlab issues finish resolving the problem. |
|
This PR has its approval and auto-merge enabled so we're just waiting on being able to run through the pipeline checks (and their success) to merge it. |
|
@spackbot run pipeline |
|
I've started that pipeline for you! |
|
@spackbot run pipeline |
|
I'm sorry, gitlab does not have your latest revision yet, I can't run that pipeline for you right now. One likely possibility is that your PR pipeline has been temporarily deferred, in which case, it is awaiting a Please check the gitlab commit status message to see if more information is available. Detailspr head: f429296, gitlab commit parents: ['3d41b71', '48666b2'] |
Adding latest PIO version 2.5.10.