Diff Detail
- Repository
- rOPS Nasqueron Operations
- Lint
Lint Passed - Unit
No Test Coverage - Branch
- strongswan
- Build Status
Buildable 6461 Build 6745: arc lint + arc unit
Event Timeline
Service part OK.
Configuration needs a better strategy to allow to provision per machine.
| _modules/credentials.py | ||
|---|---|---|
| 304 | That's handled in D3987. Perhaps you want to do an arc diff HEAD^ --update D3988 ? | |
| pillar/core/network.sls | ||
| 59 | We can use the banner format here, to be coherent with the repository style. | |
| 74 | You can provision them as a follow-up change, uncommented instead. | |
| 94 | That comment can be safely removed: the pillar name and the ike_ esp_ keys make that clear what is it. | |
| roles/core/strongswan/config.sls | ||
| 22 | Jinja syntax allows to use dots as separator to access dictionary keys. We can use it, but in that case, it's really coherent to use it everywhere. | |
| 23 | Won't really work beyond the scope of a test tunnel between those two links. You need a more flexible way to pass the info according the node. Besides, if you deploy this on both router-002 and router-003 you would get twice the config router-002 to router-003 it seems. | |
| roles/core/strongswan/files/swanctl.conf | ||
| 2 | Header missing, see https://devcentral.nasqueron.org/source/snippets/browse/main/salt/file.conf | |
| roles/core/strongswan/init.sls | ||
| 2 | Header missing | |
| 5 | That one doesn't exist in the change it seems | |
| roles/core/strongswan/files/swanctl.conf | ||
|---|---|---|
| 50 | ||
Test to deploy this final version on Complector with a test=True to see if it's still no-op.