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

Drop all CvmFS server code #113

Merged
merged 2 commits into from
Mar 18, 2021
Merged

Conversation

traylenator
Copy link
Contributor

@traylenator traylenator commented Mar 11, 2021

Remove cvmfs server code completly.

The server code is no longer used for quite some time and is stale without tests.

In addition add LICENSE file.

@traylenator traylenator changed the title Drop all CvmFS server code WIP: Drop all CvmFS server code Mar 12, 2021
@traylenator traylenator changed the title WIP: Drop all CvmFS server code Drop all CvmFS server code Mar 15, 2021
@luisfdez
Copy link
Contributor

Hi @traylenator , it does look good. The only thing that maybe we could remove as well is cvmfs-server from the default $cvmfs_yum_includepkgs in init.pp, as I think it doesn't add much value once this is done.

@traylenator traylenator merged commit 56ff19d into voxpupuli:master Mar 18, 2021
@traylenator traylenator deleted the dropserver branch March 18, 2021 15:50
traylenator added a commit to traylenator/puppet-cvmfs that referenced this pull request Mar 23, 2021
**Breaking changes:**

- Remove 'yum' string from class parameters [\voxpupuli#123](voxpupuli#123) ([traylenator](https://github.com/traylenator))
- Add type enforcement to cvmfs::mount and cvmfs::domain [\voxpupuli#122](voxpupuli#122) ([traylenator](https://github.com/traylenator))
- Drop all CvmFS server code [\voxpupuli#113](voxpupuli#113) ([traylenator](https://github.com/traylenator))

**Implemented enhancements:**

- Add CVMFS\_IPFAMILY\_PREFER config parameter [\voxpupuli#125](voxpupuli#125) ([luisfdez](https://github.com/luisfdez))
- Add ubuntu 18.04, 20.04 and debian 10 support [\voxpupuli#124](voxpupuli#124) ([traylenator](https://github.com/traylenator))
- Convert main config template to epp [\voxpupuli#118](voxpupuli#118) ([traylenator](https://github.com/traylenator))

**Fixed bugs:**

- cvmfs\_external\_url parameter to mount fixed [\voxpupuli#127](voxpupuli#127) ([traylenator](https://github.com/traylenator))
- Correct parameter name for default case [\voxpupuli#126](voxpupuli#126) ([traylenator](https://github.com/traylenator))

**Merged pull requests:**

- Update all documentation [\voxpupuli#128](voxpupuli#128) ([traylenator](https://github.com/traylenator))
- Use modern splat rather than create resources [\voxpupuli#121](voxpupuli#121) ([traylenator](https://github.com/traylenator))
- Whitespace, Automatic lint and Rubocop Fixes [\voxpupuli#114](voxpupuli#114) ([traylenator](https://github.com/traylenator))
- Increase upper version of firewall module [\voxpupuli#112](voxpupuli#112) ([sorrison](https://github.com/sorrison))
- cvmfs::install: Fact file creation must depend on Package\[cvmfs\]. [\voxpupuli#111](voxpupuli#111) ([olifre](https://github.com/olifre))
- Add UID\_MAP and GID\_MAP functionality. [\voxpupuli#110](voxpupuli#110) ([olifre](https://github.com/olifre))
- add missing package require for file ownership [\voxpupuli#104](voxpupuli#104) ([fschaer](https://github.com/fschaer))
@traylenator traylenator mentioned this pull request Mar 23, 2021
traylenator added a commit to traylenator/puppet-cvmfs that referenced this pull request Oct 8, 2021
Since the cvmfs server configuration was removed in
voxpupuli#113
the firewall and limits puppet module are no longer
used.
traylenator added a commit to traylenator/puppet-cvmfs that referenced this pull request Oct 8, 2021
Since the cvmfs server configuration was removed in
voxpupuli#113
the firewall and limits puppet module are no longer
used.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants