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

[secaudit-blocking] No installation documentation #332

Closed
puiterwijk opened this issue Aug 11, 2020 · 23 comments
Closed

[secaudit-blocking] No installation documentation #332

puiterwijk opened this issue Aug 11, 2020 · 23 comments
Labels
documentation Improvements or additions to documentation security Security issue

Comments

@puiterwijk
Copy link

puiterwijk commented Aug 11, 2020

Part of secaudit #316, blocking.

The Installation documentation is absent, and doesn't contain any useful information in how to set up a production-grade setup of noggin.

@puiterwijk puiterwijk added the security Security issue label Aug 11, 2020
@puiterwijk puiterwijk changed the title [secaudit #316] No installation documentation [secaudit] No installation documentation Aug 11, 2020
@puiterwijk puiterwijk changed the title [secaudit] No installation documentation [secaudit-blocking] No installation documentation Aug 11, 2020
@ryanlerch
Copy link
Contributor

@puiterwijk we will get working on this ASAP.

Note too, (while not documentation, i know), there is the vagrant setup that sets up a freeipa server, installs the freeipa-fas plugin to freeipa, then installs noggin.

Note too there is an in-review PR here (#326) that makes the noggin flask app behave a little better.

@puiterwijk
Copy link
Author

@ryanlerch Right. But note that if you decide the Vagrant setup is the official way of deploying, I'll have many complaints about insecure deployment practices 😀. That's why I want to see how you tell people to actually deploy it.

@nphilipp
Copy link
Member

@puiterwijk I don't think we'll document using Vagrant to deploy in production, with or without your comment 😉. Off the top of your head, are there any other gotchas? If we can avoid embarrassing ourselves upfront, I'm all for it.

@puiterwijk
Copy link
Author

puiterwijk commented Aug 13, 2020

Some of the things you really want to point out:

  • Don't use flask run, but instead use a serious HTTP server, and explain how to do so (apache/nginx with possible gunicorn behind it)
  • Do not use the global admin user: instead, I'd strongly recommend a separate noggin user for auditing/permission purposes (as part of the docs would then be "What are the minimum required permissions to grant")
  • Make very sure to change SECRET and FERNET_SECRET ([secaudit-blocking] Default configuration insecure defaults #334)

And other things like those

@Conan-Kudo
Copy link
Contributor

@nphilipp @puiterwijk Probably a good starting point for installation documentation would be the haphazard one I wrote for getting the system up and running for openSUSE infrastructure on COPR: https://copr.fedorainfracloud.org/coprs/ngompa/fedora-aaa/

@davidkirwan davidkirwan self-assigned this Oct 15, 2020
@davidkirwan davidkirwan removed their assignment Nov 13, 2020
@ryanlerch ryanlerch added the documentation Improvements or additions to documentation label Jan 12, 2021
@Cliftonz
Copy link

Cliftonz commented Jun 4, 2021

Any updates on this?

@abompard
Copy link
Member

abompard commented Jun 4, 2021

Yeah there is still no proper installation documentation, sadly. But all the steps and files we use in our Openshift deployment are publicly accessible (playbook, template). I know it's not ideal but it's there.

Noggin is deployed in Openshift using the python s2i container, which runs gunicorn. It connects to IPA with a specific user that only has the necessary permissions (setup by this playbook). The SECRET and FERNET_SECRET variables are long randomly generated strings (that are, obviously, not public).

@Conan-Kudo
Copy link
Contributor

We'll probably have classical setup information available soon too, it just takes some time to run through a setup and make it a thing.

@Cliftonz
Copy link

Cliftonz commented Jun 4, 2021

That's understandable.
I do want to ask what is the recommended setup for this.
For example, if I have 4 IPA servers internally do I want to install it on each of them? Just of the IPA servers? etc

@abompard
Copy link
Member

abompard commented Jun 4, 2021

If it's an IPA cluster, then you only need one instance of Noggin.

@Cliftonz
Copy link

This issue has been sitting out for over a year. Is there any way someone could upload a rudimentary list of steps to install this product?

We do not use Openshift and want to install this manually for our ipa cluster.

@Conan-Kudo
Copy link
Contributor

I guess I could write up a guide for installing it the traditional way with the RPMs I made of this. The main reason I haven't done it yet is that I need to finish the work to update it to the latest stable version in Fedora.

@Cliftonz
Copy link

@Conan-Kudo That would much be appreciated.
I do think you should include how to install it with the playbooks and templates too.
Do you have an estimate on how long this may take?

@Conan-Kudo
Copy link
Contributor

As I don't use Ansible much, I'm not sure I could help there, but at least I can document the manual setup process and someone can contribute Ansible stuff. As for an estimate, my priorities at the moment do not leave me a lot of time for this right now, but I'm hoping to come back to this in mid-December.

@Cliftonz
Copy link

Cliftonz commented Nov 27, 2021 via email

@Cliftonz
Copy link

@Conan-Kudo Any updates?

@Conan-Kudo
Copy link
Contributor

Life happened the past few months, but I'm coming back to this.

@Cliftonz
Copy link

@Conan-Kudo Did life get in the way again?

@Conan-Kudo
Copy link
Contributor

@Cliftonz actually, in the process of testing it, I discovered that the deployment was broken and I'm trying to figure out why... 😕

@Cliftonz
Copy link

Awesome!

@Conan-Kudo
Copy link
Contributor

I've made some progress on this, I'm having @jonathanspw test out my draft before submitting it upstream.

@Cliftonz
Copy link

Cliftonz commented Oct 2, 2023

@Conan-Kudo updates?

abompard added a commit that referenced this issue Oct 4, 2023
Source: https://src.fedoraproject.org/rpms/noggin/tree/main

Thanks to Neal on this one.

Fixes: #1289 #332

Signed-off-by: Aurélien Bompard <aurelien@bompard.org>
@Cliftonz
Copy link

@abompard @Conan-Kudo I think this be closed now, correct?

@ryanlerch ryanlerch added this to AAA Jul 19, 2024
@ryanlerch ryanlerch moved this to Backlog in AAA Jul 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation security Security issue
Projects
No open projects
Status: Backlog
Development

No branches or pull requests

7 participants