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

CHTC on OSPool #4174

Merged
merged 8 commits into from
Oct 2, 2024
Merged

Conversation

CannonLock
Copy link
Collaborator

Initial PR for registering all the CHTC projects on the OSPool.

Current errors that @ChristinaLK should speak to:

[WARNING] Project is active in OSPool but not found in mapping_all: ece_mawst
[WARNING] Project is active in OSPool but not found in mapping_all: glow
[WARNING] Project is active in OSPool but not found in mapping_all: uwmadison_statistics_katzfuss
[WARNING] Project is active in OSPool but not found in mapping_all: chtc-visitors
[WARNING] Defaulted to PI Name Anthony Gitter but there were multiple options: ['Anthony Gitter', 'Christopher Magnano']
[WARNING] Project is active in OSPool but not found in mapping_all: uwmadison_chtcfellowship_2024
[WARNING] Project is active in OSPool but not found in mapping_all: ece_hu
[WARNING] Project is active in OSPool but not found in mapping_all: uwmadison_waisman_dean

These projects that report warnings for being not found in mapping all can be added manually or added to mapping_all.csv.

Otherwise a facilitator should run through and add the correct FOSID's to all of these projects.

Copy link
Collaborator

@osg-bot osg-bot left a comment

Choose a reason for hiding this comment

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

Greetings @CannonLock,

Thank you for your pull request.

Your requested changes add a new organization or project ID to a project file; this requires manual review from OSG Staff.

-- OSG-BOT 🚔


Output from the automerge check script:

Commit is not eligible for auto-merge:
 - File 'projects/BMI_Craven.yaml' is not a downtime file.
 - File 'projects/BMI_Gitter.yaml' is not a downtime file.
 - File 'projects/BMI_Hu.yaml' is not a downtime file.
 - File 'projects/BMI_Tang.yaml' is not a downtime file.
 - File 'projects/BiochemSenes.yaml' is not a downtime file.
 - File 'projects/Biochemistry_Romero.yaml' is not a downtime file.
 - File 'projects/CHTC.yaml' is not a downtime file.
 - File 'projects/CS_Albargouthi.yaml' is not a downtime file.
 - File 'projects/ChemistrySchmidt.yaml' is not a downtime file.
 - File 'projects/ECE_Lee.yaml' is not a downtime file.
 - File 'projects/ECE_Vinayak.yaml' is not a downtime file.
 - File 'projects/GeneticsHittinger.yaml' is not a downtime file.
 - File 'projects/GeneticsPool.yaml' is not a downtime file.
 - File 'projects/Genetics_Payseur.yaml' is not a downtime file.
 - File 'projects/Pharmacy_Kwan.yaml' is not a downtime file.
 - File 'projects/Physics_Joynt.yaml' is not a downtime file.
 - File 'projects/Physics_Soley.yaml' is not a downtime file.
 - File 'projects/Psychology_Curtin.yaml' is not a downtime file.
 - File 'projects/SmallMolecule_Hoffman.yaml' is not a downtime file.
 - File 'projects/UWMadison_Gitter.yaml' is not a downtime file.
 - Unrecognized InstitutionID in project(s) projects/UWMadison_Gitter.yaml. See https://topology-institutions.osg-htc.org for known ID list.


Copy link
Contributor

@ChristinaLK ChristinaLK left a comment

Choose a reason for hiding this comment

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

@brianhlin @matyasselmeci I think this is Good Enough ™️ to merge.

@CannonLock
Copy link
Collaborator Author

@ChristinaLK

Want to get these resolved pre or post merge?

[WARNING] Project is active in OSPool but not found in mapping_all: ece_mawst
[WARNING] Project is active in OSPool but not found in mapping_all: glow
[WARNING] Project is active in OSPool but not found in mapping_all: uwmadison_statistics_katzfuss
[WARNING] Project is active in OSPool but not found in mapping_all: chtc-visitors
[WARNING] Project is active in OSPool but not found in mapping_all: uwmadison_chtcfellowship_2024
[WARNING] Project is active in OSPool but not found in mapping_all: ece_hu
[WARNING] Project is active in OSPool but not found in mapping_all: uwmadison_waisman_dean

Copy link
Collaborator

@matyasselmeci matyasselmeci left a comment

Choose a reason for hiding this comment

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

LGTM; let me know if you want to make any changes.

@CannonLock
Copy link
Collaborator Author

CannonLock commented Oct 2, 2024

@ChristinaLK confirmed getting this set merged now.

@matyasselmeci Mind doing the honor?

@matyasselmeci matyasselmeci merged commit 5bf609e into opensciencegrid:master Oct 2, 2024
3 checks passed
@matyasselmeci
Copy link
Collaborator

big inviting green button... clicked!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants