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

Bugs noticed in a SingularityCE PR imported to Apptainer #3494

Open
DrDaveD opened this issue Jan 21, 2025 · 1 comment · May be fixed by #3506
Open

Bugs noticed in a SingularityCE PR imported to Apptainer #3494

DrDaveD opened this issue Jan 21, 2025 · 1 comment · May be fixed by #3506
Labels
bug Something isn't working

Comments

@DrDaveD
Copy link
Contributor

DrDaveD commented Jan 21, 2025

FYI I noticed while working on apptainer/apptainer#2712 that

  • At least the instanceDbusXdg e2e tests added in cgroups: rework checks for cgroup support #2569 are missing a call to e2e.WithRootlessEnv() (or some other way of importing the default environment) and so never actually get the cgroups v2-relevant environment variables applied. The tests happen to pass because the first two only cause debug messages and the second two always print out warnings for both variables before failing.
  • The else/debug case in the cgroups.IsCgroup2UnifiedMode check in CanUseCgroups() is copy-pasted from the !systemd check, giving the wrong debug message for instances when cgroups v2 unified mode is not available.
@DrDaveD DrDaveD added the bug Something isn't working label Jan 21, 2025
@dtrudg
Copy link
Member

dtrudg commented Jan 22, 2025

Thanks for the info @DrDaveD

Much appreciated.

dtrudg added a commit to dtrudg/singularity that referenced this issue Feb 3, 2025
Ensure that action/instanceDbusXdg e2e-tests are correctly checking
success / failure with the modified XDG env vars that are expected.

Correct an incorrect debug message from  copy/paste error.

Fixes sylabs#3494
dtrudg added a commit to dtrudg/singularity that referenced this issue Feb 3, 2025
Ensure that action/instanceDbusXdg e2e-tests are correctly checking
success / failure with the modified XDG env vars that are expected.

Correct an incorrect debug message from  copy/paste error.

Fixes sylabs#3494
dtrudg added a commit to dtrudg/singularity that referenced this issue Feb 3, 2025
Ensure that action/instanceDbusXdg e2e-tests are correctly checking
success / failure with the modified XDG env vars that are expected.

Correct an incorrect debug message from copy/paste error.

If the starter fails when starting an instance, output the starter
StdErr to the terminal StdErr - previously we were writing started
StdErr to terminal StdOut.

Fixes sylabs#3494
dtrudg added a commit to dtrudg/singularity that referenced this issue Feb 3, 2025
Ensure that action/instanceDbusXdg e2e-tests are correctly checking
success / failure with the modified XDG env vars that are expected.

Correct an incorrect debug message from copy/paste error.

If the starter fails when starting an instance, output the starter
StdErr to the terminal StdErr - previously we were writing started
StdErr to terminal StdOut.

Fixes sylabs#3494
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants