Page MenuHomeDevCentral

Install netbox on new windriver
Needs ReviewPublic

Authored by DorianWinty on Sep 27 2024, 22:18.
Tags
None
Referenced Files
F3923987: D3469.id8943.diff
Sat, Dec 21, 11:01
F3923933: D3469.diff
Sat, Dec 21, 10:40
Unknown Object (File)
Mon, Dec 9, 20:25
Unknown Object (File)
Mon, Dec 9, 17:40
Unknown Object (File)
Fri, Dec 6, 15:54
Unknown Object (File)
Fri, Dec 6, 13:55
Unknown Object (File)
Tue, Dec 3, 08:33
Unknown Object (File)
Tue, Dec 3, 08:33
Subscribers
None

Details

Reviewers
dereckson

Diff Detail

Repository
rOPS Nasqueron Operations
Lint
Lint Errors
SeverityLocationCodeMessage
Errorroles/netbox/netbox/files/configuration.py:23E501flake8 E501
Errorroles/netbox/netbox/files/configuration.py:24E501flake8 E501
Errorroles/netbox/netbox/files/configuration.py:29E501flake8 E501
Errorroles/netbox/netbox/files/configuration.py:40E501flake8 E501
Errorroles/netbox/netbox/files/configuration.py:41E501flake8 E501
Errorroles/netbox/netbox/files/configuration.py:47E501flake8 E501
Errorroles/netbox/netbox/files/configuration.py:57E501flake8 E501
Errorroles/netbox/netbox/files/configuration.py:63E501flake8 E501
Errorroles/netbox/netbox/files/configuration.py:73E501flake8 E501
Errorroles/netbox/netbox/files/configuration.py:78E501flake8 E501
Errorroles/netbox/netbox/files/configuration.py:79E501flake8 E501
Errorroles/netbox/netbox/files/configuration.py:91E501flake8 E501
Errorroles/netbox/netbox/files/configuration.py:100E501flake8 E501
Errorroles/netbox/netbox/files/configuration.py:101E501flake8 E501
Errorroles/netbox/netbox/files/configuration.py:111E501flake8 E501
Errorroles/netbox/netbox/files/configuration.py:115E501flake8 E501
Errorroles/netbox/netbox/files/configuration.py:116E501flake8 E501
Errorroles/netbox/netbox/files/configuration.py:117E501flake8 E501
Errorroles/netbox/netbox/files/configuration.py:126E501flake8 E501
Errorroles/netbox/netbox/files/configuration.py:127E501flake8 E501
Errorroles/netbox/netbox/files/configuration.py:146E501flake8 E501
Errorroles/netbox/netbox/files/configuration.py:147E501flake8 E501
Errorroles/netbox/netbox/files/configuration.py:154E501flake8 E501
Errorroles/netbox/netbox/files/configuration.py:160E501flake8 E501
Errorroles/netbox/netbox/files/configuration.py:164E501flake8 E501
Unit
No Test Coverage
Branch
netboxInstall
Build Status
Buildable 5519
Build 5800: arc lint + arc unit

Event Timeline

DorianWinty created this revision.
dereckson requested changes to this revision.Sep 29 2024, 13:10

Good for me excepted for UIDs/GIDs

GIDs
15

Sorted by numerical order

UIDs
11

Should match uid, so 1001

roles/netbox/netbox/software.sls
18

Open question:

Should we declare software versions and hashes directly in roles or use pillar to store that information?

I don't have a definite opinion on that question and generally put this kind of information directly in roles/.

Pillar could simply be:

netbox:
  release:
     tag: v3.7.1
     hash: 97ea9106b6d29e2568c4e9c395013ca015ba7521029e8c907b6aa515dd62649a
This revision now requires changes to proceed.Sep 29 2024, 13:10
DorianWinty marked 3 inline comments as done.

edit following the commants

dereckson requested changes to this revision.Sep 29 2024, 21:34
dereckson added inline comments.
roles/netbox/netbox/files/gunicorn.py
15

New server CPU is 6 core, 12 threads

To avoid to read sysctl kern.sched.topology_spec, probably the easiest way to get this value is to use Salt grains.

No need to divide by 2 as it gives the number of CPU threads, not of core.

18

That's OS-level thread, not CPU thread, so we can keep this value.

This revision now requires changes to proceed.Sep 29 2024, 21:34
roles/netbox/netbox/files/rc/netbox
46

Only one =

DorianWinty marked 2 inline comments as done.

update following comments