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

Refactor manifests #798

Closed
wants to merge 5 commits into from
Closed

Refactor manifests #798

wants to merge 5 commits into from

Conversation

dhoppe
Copy link
Member

@dhoppe dhoppe commented May 3, 2019

The following classes still have to be refactored:

  • config.pp
  • server.pp
  • repo/apt.pp
  • repo/rhel.pp

The classes apt.pp and rhel.pp depend on pr #797.

I also need to know which Mirror we want to use as default and change the GPG key. I would like to use Bintray, because it provides packages for Erlang.

Because of the many changes to the code we will have to release a major release anyway.

dhoppe added 4 commits May 3, 2019 13:46
  * Update header for puppet-strings
  * Assert that class is private
  * Remove useless variables
  * Update header for puppet-strings
  * Assert that class is private
  * Remove useless variables
  * Update header for puppet-strings
  * Assert that class is private
  * Update header for puppet-strings
  * Assert that class is private
  * Remove useless variables
  * Use groupname instead of gid
  * Make class more readable
@dhoppe dhoppe added the needs-work not ready to merge just yet label May 3, 2019
@wyardley
Copy link
Contributor

wyardley commented May 3, 2019

#781 has some more info about the keys. I like the suggestion to just bake the files directly into the module; this is probably the most secure option.

@wyardley
Copy link
Contributor

wyardley commented May 3, 2019

Also, I don’t have a preference on mirror, but packagecloud has their erlang packages too:
https://packagecloud.io/rabbitmq/erlang

@dhoppe
Copy link
Member Author

dhoppe commented May 3, 2019

Also, I don’t have a preference on mirror, but packagecloud has their erlang packages too:
https://packagecloud.io/rabbitmq/erlang

Yes, but they only provide Erlang packages for RedHat/CentOS.

@dhoppe
Copy link
Member Author

dhoppe commented May 3, 2019

#781 has some more info about the keys. I like the suggestion to just bake the files directly into the module; this is probably the most secure option.

Yes, but I do not know if we just need to provide the latest GPG key. My understanding was that they resigned all packages with the current GPG key.

@bastelfreak
Copy link
Member

@dhoppe can you please rebase?

@wyardley
Copy link
Contributor

wyardley commented Aug 1, 2019

@dhoppe do you want to work on this some more? I probably should cut a major release soon, and it would be nice to include these changes if you're able to update them.

@wyardley wyardley mentioned this pull request Aug 14, 2019
@vox-pupuli-tasks
Copy link

Dear @dhoppe, thanks for the PR!

This is Vox Pupuli Tasks, your friendly Vox Pupuli GitHub Bot. I noticed that your pull request contains merge conflict. Can you please rebase?

You can find my sourcecode at voxpupuli/vox-pupuli-tasks

@wyardley
Copy link
Contributor

wyardley commented May 6, 2021

@dhoppe I'm going to close this one for now; feel free to reopen / revisit when you're ready to work on this again.

@wyardley wyardley closed this May 6, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants