-
Notifications
You must be signed in to change notification settings - Fork 260
* Update UConn osg resources #5056
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
Conversation
osg-bot
left a comment
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.
Greetings @rjones30,
Thank you for your pull request.
Your requested changes affect one or more downtime files, but are not eligible for downtime automerge; OSG Staff will need to manually review this PR.
-- OSG-BOT 🔍
Output from the automerge downtime check script:
Old Downtime 1734780663 modified for resource 'UConn-HPC_StashCache_origin'
Old Downtime 2151043084 modified for resource 'UConn-HPC_StashCache_origin'
Old Downtime 2146091711 modified for resource 'UConn-HPC_CE'
Commit is not eligible for auto-merge:
- File 'topology/University of Connecticut/UConn - cedar/SITE.yaml' is not a downtime file.
- File 'topology/University of Connecticut/UConn-HPC/SITE.yaml' is not a downtime file.
- File 'topology/University of Connecticut/UConn-HPC/UConn-HPC.yaml' is not a downtime file.
- File 'topology/University of Connecticut/UConn-OSG/SITE.yaml' is not a downtime file.
- File 'topology/University of Connecticut/UConn-OSG/UConn-OSG.yaml' is not a downtime file.
- File 'topology/University of Connecticut/UConn-Xanadu/SITE.yaml' is not a downtime file.
- File 'topology/University of Connecticut/UConn-Xanadu/UConn-Xanadu.yaml' is not a downtime file.
|
@rjones30 you've marked this as "do-not-merge". What's left to be done here before we can review and merge? |
UConn-HPC and mark it Active: false. This required that I assign a new name UConn-HTC_StashCache_origin for the moved OSDF origin that was formerly named UConn-HPC_StashCache_origin, served from a different IP domain. [rtj]
the former assignment of this same ID to UConn-HPC_StashCache_origin when this same storage resource was located elsewhere in IP space.
|
Richard, I looked through the updates and they seem fine to me, but I am not the final arbiter of what goes into Topology. However, I removed all of the “do not merge” bits and will follow up internally to get a final review and hopefully a merge. Thank you for all your effort in helping to clean up our Topology database! — Tim C. |
matyasselmeci
left a comment
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.
Two small changes but otherwise good.
|
@rjones30, there is nothing for you to do here — the PR is basically ready, and we are just waiting on GitHub to complete automated checks, then we plan to merge. However, GitHub is having some issues right at the moment. — Tim C. |
Align with recent resource hardware consolidation - Richard Jones