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.

Sign up for GitHub

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

Jump to bottom

Do not for call cvmfsexec script if not necessary #308

Closed
mmascher opened this issue Jun 22, 2023 · 0 comments · Fixed by #309
Closed

Do not for call cvmfsexec script if not necessary #308

mmascher opened this issue Jun 22, 2023 · 0 comments · Fixed by #309
Assignees
@mmascher
Labels
BUG For BUGS cms CMS stakeholder factory for affected component factoryops Factory Operations stakeholder

Comments

@mmascher
Copy link
Contributor

Describe the bug
Not really a bug, but when reconfiguring the ITB dev factory I got an out of memory. Turned out that since subprocess uses fork then the whole process memory is duplicated. And since the test machine is really small the memory is not enough.

I'd suggest to avoid doing the subprocess call if not necessary (cvmfs has been rutned off).

  • Stakeholders: CMS, FactoryOps
  • Components: factory
@mmascher mmascher self-assigned this Jun 22, 2023
@github-actions github-actions bot added BUG For BUGS cms CMS stakeholder factory for affected component factoryops Factory Operations stakeholder labels Jun 22, 2023
mmascher added a commit to mmascher/glideinwms that referenced this issue Jun 22, 2023
@mmascher
Do not for call cvmfsexec script if not necessary. Fixes glideinWMS#308
446f528
mmascher added a commit to mmascher/glideinwms that referenced this issue Jun 22, 2023
@mmascher
Do not for call cvmfsexec script if not necessary. Fixes glideinWMS#308
9bae372
@namrathaurs namrathaurs linked a pull request Jun 30, 2023 that will close this issue
Avoid invocation of cvmfsexec distros building if not necessary #309
Merged
@namrathaurs namrathaurs mentioned this issue Jun 30, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mmascher mmascher

Labels
BUG For BUGS cms CMS stakeholder factory for affected component factoryops Factory Operations stakeholder
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Avoid invocation of cvmfsexec distros building if not necessary mmascher/glideinwms
1 participant
@mmascher