Page MenuHomeSoftware Heritage

Add netbox inventory tool support
ClosedPublic

Authored by vsellier on Thu, Sep 24, 9:41 AM.

Details

Diff Detail

Repository
rSPSITE puppet-swh-site
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

vsellier created this revision.Thu, Sep 24, 9:41 AM
vsellier planned changes to this revision.Thu, Sep 24, 9:41 AM
vsellier updated this revision to Diff 14238.Thu, Sep 24, 5:29 PM
  • add apache and redis configurations
  • Use the swh forge's redis repository
  • use role::swh_server as base roleto activate the backup
vsellier updated this revision to Diff 14268.Fri, Sep 25, 12:56 PM
  • Use the archive module to install the tgz
  • Declare missing dependencies to fix the initial puppet run
  • Install dir is now read-only for the netbox user
  • Restore the whitespaces on the cassandra configuration
vsellier updated this revision to Diff 14281.Fri, Sep 25, 3:24 PM

isolate the reverse_proxy configuration in a dedicated profile

vsellier added a subscriber: olasd.EditedFri, Sep 25, 3:31 PM

@olasd All the points we have checked yesterday should be addressed.

I gave up on using the swh_gunicorn module to configure the service because there are some hardcoded behaviors that don't match with the nedbox configuration (for example the location of the gunicorn executable, or the usage of a virtualenv not supported)

I'm not really sure of the vhost configuration but I can't test it locally due to the certificate management. It will befine tuned on the real server.

vsellier updated this revision to Diff 14356.Mon, Sep 28, 3:35 PM
  • rebase
  • add the vagrant configuration
ardumont added inline comments.
site-modules/profile/templates/netbox/configuration.py.erb
34

Should use the redis_host and redis_port variables to avoid scratching heads later when the default values change.

45

same, i guess.

ardumont accepted this revision.Wed, Sep 30, 9:10 AM

looks good to me, one remark inline.

This revision is now accepted and ready to land.Wed, Sep 30, 9:10 AM
vsellier updated this revision to Diff 14444.Wed, Sep 30, 12:59 PM

Use redis's configuration properties in the configuration template

vsellier marked 2 inline comments as done.Wed, Sep 30, 1:00 PM
vsellier added inline comments.
site-modules/profile/templates/netbox/configuration.py.erb
34

good catch, it's fixed

vsellier updated this revision to Diff 14515.Thu, Oct 1, 7:02 PM
vsellier marked an inline comment as done.

Uppdate to match the real server name

This revision was automatically updated to reflect the committed changes.