Skip to content

parallelio: new version#35553

Merged
tldahlgren merged 3 commits intospack:developfrom
theurich:parallelio_2_5_10
Mar 8, 2023
Merged

parallelio: new version#35553
tldahlgren merged 3 commits intospack:developfrom
theurich:parallelio_2_5_10

Conversation

@theurich
Copy link
Copy Markdown
Contributor

Adding latest PIO version 2.5.10.

@theurich
Copy link
Copy Markdown
Contributor Author

@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 jedwards4b self-requested a review February 20, 2023 17:42
Copy link
Copy Markdown
Contributor

@jedwards4b jedwards4b left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think that you need to add me as a reviewer to notify me.

@theurich
Copy link
Copy Markdown
Contributor Author

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.

jedwards4b
jedwards4b previously approved these changes Feb 20, 2023
@jedwards4b
Copy link
Copy Markdown
Contributor

merge will be blocked until one of the spack reviewers looks at it.

@theurich
Copy link
Copy Markdown
Contributor Author

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 ...

tldahlgren
tldahlgren previously approved these changes Feb 21, 2023
Copy link
Copy Markdown
Contributor

@tldahlgren tldahlgren left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Confirmed the sha256.

@tldahlgren
Copy link
Copy Markdown
Contributor

@spackbot run pipeline

@spackbot-app
Copy link
Copy Markdown

spackbot-app bot commented Feb 21, 2023

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 develop pipeline, and will be run when that finishes.

Please check the gitlab commit status message to see if more information is available.

Details
Unexpected response from gitlab: {'message': '404 Commit Not Found'}

@tldahlgren tldahlgren enabled auto-merge (squash) February 21, 2023 20:13
@theurich
Copy link
Copy Markdown
Contributor Author

@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.

@tldahlgren
Copy link
Copy Markdown
Contributor

@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.

@theurich
Copy link
Copy Markdown
Contributor Author

Okay, updated the branch against develop. Let's hope it helps to move it forward.

@alalazo alalazo dismissed stale reviews from tldahlgren and jedwards4b via a56feb4 February 21, 2023 20:53
@tldahlgren
Copy link
Copy Markdown
Contributor

Okay, updated the branch against develop. Let's hope it helps to move it forward.

🤞

@theurich theurich requested a review from jedwards4b February 22, 2023 19:59
@theurich
Copy link
Copy Markdown
Contributor Author

@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!

@jedwards4b
Copy link
Copy Markdown
Contributor

It's normal - consider that the repo currently has over 600 open PRs.

@alalazo
Copy link
Copy Markdown
Member

alalazo commented Feb 23, 2023

@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.

@tldahlgren
Copy link
Copy Markdown
Contributor

@spackbot run pipeline

@spackbot-app
Copy link
Copy Markdown

spackbot-app bot commented Feb 23, 2023

I've started that pipeline for you!

@theurich
Copy link
Copy Markdown
Contributor Author

Thanks everyone... I am learning.
@tldahlgren - should/could I have given the request to run the pipeline?

@tldahlgren
Copy link
Copy Markdown
Contributor

Thanks everyone... I am learning.

You're welcome.

@tldahlgren - should/could I have given the request to run the pipeline?

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.

@tldahlgren
Copy link
Copy Markdown
Contributor

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.

@alalazo
Copy link
Copy Markdown
Member

alalazo commented Mar 1, 2023

@spackbot run pipeline

@spackbot-app
Copy link
Copy Markdown

spackbot-app bot commented Mar 1, 2023

I've started that pipeline for you!

@theurich
Copy link
Copy Markdown
Contributor Author

theurich commented Mar 6, 2023

@spackbot run pipeline

@spackbot-app
Copy link
Copy Markdown

spackbot-app bot commented Mar 6, 2023

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 develop pipeline, and will be run when that finishes.

Please check the gitlab commit status message to see if more information is available.

Details
pr head: f429296, gitlab commit parents: ['3d41b71', '48666b2']

@tldahlgren tldahlgren merged commit 87c21a5 into spack:develop Mar 8, 2023
jmcarcell pushed a commit to key4hep/spack that referenced this pull request Apr 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants