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

New client packages (also tar now) that don't use the config repo #70

Merged
merged 44 commits into from
Mar 2, 2021
Merged

New client packages (also tar now) that don't use the config repo #70

merged 44 commits into from
Mar 2, 2021

Conversation

bedroge
Copy link
Collaborator

@bedroge bedroge commented Feb 10, 2021 •
edited
Loading

Closes #68 and partially #59.

See https://github.com/bedroge/filesystem-layer/releases/tag/v0.3.0 for an example of a release that was generated using the code of this branch.

bedroge added 30 commits February 8, 2021 16:02
@bedroge
prepare new client package without config repo
c616e8a
@bedroge
enable ci on this branch
40c5cae
@bedroge
enable geoapi
6df5d70
@bedroge
use config repo domain for now
353f1bf
@bedroge
use existing config in group vars
e745d59
@bedroge
add public keys, remove new part
2628f25
@bedroge
remove whitespaces
381bec0
@bedroge
change name of task
207c201
@bedroge
correct name of pilot key
aa2b10b
@bedroge
Let CI test access to the pilot repo instead of the cvmfs-config repo
8c3ffa6
@bedroge
build tar package
8f4ef56
@bedroge
fix the ls test, add test for tarball
c572367
@bedroge
fix macos test
e6c345c
@bedroge
remove current test branch for building packages
efe57f0
@bedroge
comment the eessi config repo
5d82d36
@bedroge
comment config repo for stratum 0/1
f767fc8
@bedroge
still include the old cvmfs-config repo for the CI
2e446af
@bedroge
still include the old cvmfs-config repo for the CI
7fde248
@bedroge
dont run the cvmfs-config playbook in CI
1e6156e
@bedroge
replace cvmfs-config by ci repo
35f7739
@bedroge
escape quotes
458b7ef
@bedroge
use heredoc instead of echo
809d9d5
@bedroge
use eq instead of equalto
f4934f3
@bedroge
use sameas instead of eq
f89ebb4
@bedroge
dont select only the repo name
02f93ac
@bedroge
add pubkey for ci repo
787d438
@bedroge
add note about PR galaxyproject/ansible-cvmfs#32
29defa4
@bedroge
don't use config repo for clients
3f0d208
@bedroge
configure clients statically (no config repo)
5d606af
@bedroge
remove config repo lines
c008c1b
@bedroge
Copy link
Collaborator Author

bedroge commented Feb 15, 2021

Also closes #69 , since this package won't depend on the cvmfs-config repo.

@bedroge bedroge linked an issue Feb 15, 2021 that may be closed by this pull request
Add 'Provides' (and 'Conflicts'?) flags to our config packages #69
Closed
@bedroge
Copy link
Collaborator Author

bedroge commented Feb 15, 2021

I asked for a bit of advice on the CVMFS forum:
https://cernvm-forum.cern.ch/t/best-practices-for-client-configuration-packages-without-config-repo/47

As discussed there, I'm setting CVMFS_HTTP_PROXY=DIRECT in case the client hasn't set this parameter yet, but does have CVMFS_CLIENT_PROFILE=single.

Note that I'm still including multiple public keys here, since not all repos share the same masterkey yet and this still requires a change in the Ansible role (for which I opened a PR).

@bedroge bedroge marked this pull request as ready for review February 15, 2021 09:15
Base automatically changed from master to main February 18, 2021 15:12
boegel
boegel reviewed Mar 2, 2021
View reviewed changes
README.md Show resolved Hide resolved
@bedroge bedroge mentioned this pull request Mar 2, 2021
Open
boegel
boegel reviewed Mar 2, 2021
View reviewed changes
inventory/group_vars/all.yml Outdated Show resolved Hide resolved
boegel
boegel approved these changes Mar 2, 2021
View reviewed changes
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

@boegel boegel merged commit 80ec3ba into EESSI:main Mar 2, 2021
@bedroge bedroge deleted the remove_config_repo branch March 2, 2021 19:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@boegel boegel boegel approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Add 'Provides' (and 'Conflicts'?) flags to our config packages Provide tarball with client configuration
2 participants
@bedroge @boegel