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

Site fixes in prod-consistency-jobs.yaml #324

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

mrguilima
Copy link
Contributor

The changes fix some issues in three sites, as manually tested in the consistency pod:

  • T2_TR_METU: fix server path
  • T2_IN_TIFR: fix server port
  • T2_RU_INR: fix server port

Copy link
Contributor

@dynamic-entropy dynamic-entropy left a comment

Choose a reason for hiding this comment

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

Hello @mrguilima

Can you please use this to configure sites? And comment out the config in git completely?
https://github.com/dmwm/cms_consistency/tree/master/site_cmp3#parameters-controlled-by-site-admin

Also, it would be nice if you are already maintaining a checklist or similar of sites that have been corrected including reference to any communication or agreement that happened with the site admins regarding these configuration.
This would help others who would look at this config in the future.

According to me, an even better thing to do, would be to take care of this issue first, dmwm/CMSRucio#820

Then you can ask the site admins to correct the port. This would then require communication with site support and getting the values correct.
This is just a suggestion, you may take whichever approach you prefer.

@mrguilima
Copy link
Contributor Author

mrguilima commented Aug 7, 2024

I tried to get information from rucio-admin about the sites I've changed:

jlima@lxplus992 .globus> rucio-admin rse  info T2_FI_HIP | grep CE
  CE_config.server: hip-cms-se.csc.fi:1094
jlima@lxplus992 .globus> rucio-admin rse  info T2_BR_UERJ | grep CE
jlima@lxplus992 .globus> rucio-admin rse  info T2_IN_TIFR | grep CE
jlima@lxplus992 .globus> rucio-admin rse  info T2_RU_INR | grep CE
jlima@lxplus992 .globus> rucio-admin rse  info T2_TR_METU | grep CE

but there is no relevant info available.

Are you suggesting me to use rucio-admin to set the parameters I need, even though they are not currently defined?

@dynamic-entropy
Copy link
Contributor

Yes!

@mrguilima
Copy link
Contributor Author

So far, except for the T2_BR_UERJ site, there has been no communication with site admins to obtain information.
The changes proposed have been found by trial-and-error, and tested manually to confirm that they fix the site connection issues, and then the site entries in the CE monitoring page.

Part of the reason why I did not contact site admins is because I don't know who they are or how to contact them. Can you suggest where to find out who the site admin(s) are for each site, and contact information?

@mrguilima
Copy link
Contributor Author

I don't have necessary permission to set site attributes:

jlima@lxplus969 rucio> rucio-admin rse set-attribute --rse T2_BR_UERJ --key CE_config.server --value xrootd2.hepgrid.uerj.br:1094
Access to the requested resource denied.
Details: Account jlima can not add RSE attributes
This error is a permission issue. You cannot run this command with your account.

@dynamic-entropy
Copy link
Contributor

@eachristgr can you please add an admin tag on @mrguilima account?

@dynamic-entropy
Copy link
Contributor

@mrguilima Can you please comment out / remove the lines that you are trying to modify in this PR after you have changed the RSE config?

The aim is to reach 0 lines in the cc config for rses in flux.

@eachristgr
Copy link
Contributor

@eachristgr can you please add an admin tag on @mrguilima account?

I do not have the required permissions to do so.

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.

3 participants