Details
- Reviewers
dereckson
Diff Detail
- Repository
- rOPS Nasqueron Operations
- Lint
Lint Passed - Unit
No Test Coverage - Branch
- strongswan
- Build Status
Buildable 6390 Build 6674: arc lint + arc unit
Event Timeline
Service part OK.
Configuration needs a better strategy to allow to provision per machine.
| _modules/credentials.py | ||
|---|---|---|
| 304 ↗ | (On Diff #10353) | That's handled in D3987. Perhaps you want to do an arc diff HEAD^ --update D3988 ? |
| pillar/core/network.sls | ||
| 58 | We can use the banner format here, to be coherent with the repository style. | |
| 73 | You can provision them as a follow-up change, uncommented instead. | |
| 93 | 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 | ||
| 1 ↗ | (On Diff #10353) | Header missing |
| 4 ↗ | (On Diff #10353) | That one doesn't exist in the change it seems |