Skip to content

Hide magic attributes in CVMFS configuration of EESSI containers + update to CernVM-FS 2.9.4 #125

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

Merged
merged 11 commits into from
Sep 2, 2022

Conversation

bedroge
Copy link
Collaborator

@bedroge bedroge commented Jul 21, 2022

See #110. Also bumped the version of CVMFS to 2.9.4.

ocaisa
ocaisa previously approved these changes Jul 21, 2022
Copy link
Member

@ocaisa ocaisa left a comment

Choose a reason for hiding this comment

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

LGTM

@bedroge
Copy link
Collaborator Author

bedroge commented Jul 21, 2022

Oh, right, I have to spin up our VM with self-hosted runners for some actions...

@bedroge
Copy link
Collaborator Author

bedroge commented Jul 25, 2022

Not sure why that check is failing, will check again after my vacation.

@ocaisa
Copy link
Member

ocaisa commented Aug 4, 2022

Should we consider adding this setting to the recommended configuration in the README...and are there other places we should inject it?

@bedroge
Copy link
Collaborator Author

bedroge commented Aug 11, 2022

Should we consider adding this setting to the recommended configuration in the README...and are there other places we should inject it?

That probably makes sense, let's add that to issue #110?

I've fixed the container builds on ppc64le. Also included a fix for the client containers, which had a broken fuse-overlayfs (it was building a binary, but then it also tried to download a non-existing binary, which resulted in a file containing 404 not found).

@boegel boegel changed the title Hide magic attributes in CVMFS configuration of EESSI containers Hide magic attributes in CVMFS configuration of EESSI containers + update to CernVM-FS 2.9.4 Sep 1, 2022
Copy link
Contributor

@boegel boegel left a comment

Choose a reason for hiding this comment

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

lgtm

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