Page MenuHomeDevCentral
Differential D3660

Fix docker_containers app_port/host test
ClosedPublic
Actions

Authored by dereckson on Sep 11 2025, 22:58.
Tags
None
Referenced Files
F12332536: D3660.id.diff
Sat, Oct 25, 06:49
Unknown Object (File)
Fri, Oct 24, 13:15
Unknown Object (File)
Wed, Oct 22, 22:15
Unknown Object (File)
Wed, Oct 22, 19:49
Unknown Object (File)
Tue, Oct 21, 04:36
Unknown Object (File)
Mon, Oct 20, 11:15
Unknown Object (File)
Sat, Oct 18, 22:52
Unknown Object (File)
Sat, Oct 18, 16:16
View All 44 Files
Subscribers
None

Details

Reviewers
DorianWinty
Maniphest Tasks
T2123: Fix tests for operations repository
Commits
rOPS3bae0bccf2aa: Fix docker_containers app_port/host test
Summary

Instead of reading the pillar from paas/docker.sls,
with Salt Tower, we need to read every .sls files
in pillar/paas/docker/.

The new structure contains directly the services,
so we skip the "by node" step when reading the dictionary.

This is a follow-up for bbc48f9eb26d.

Ref T2123

Test Plan

(cd _tests && python3 -m unittest pillar/paas/test_docker.py)

Diff Detail

Repository
rOPS Nasqueron Operations
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

dereckson requested review of this revision.Sep 11 2025, 22:58
dereckson created this revision.
Harbormaster completed remote builds in B5863: Diff 9483.Sep 11 2025, 22:58
DorianWinty accepted this revision.Mon, Oct 13, 14:40
This revision is now accepted and ready to land.Mon, Oct 13, 14:40
Closed by commit rOPS3bae0bccf2aa: Fix docker_containers app_port/host test (authored by dereckson). · Explain WhyTue, Oct 14, 17:03
This revision was automatically updated to reflect the committed changes.
dereckson added a commit: rOPS3bae0bccf2aa: Fix docker_containers app_port/host test.

Revision Contents
Changeset List

PathSize
_tests/
pillar/
paas/
43 lines

Diff 9718

View Options

_tests/pillar/paas/test_docker.py

Loading...
Nasqueron DevCentral · If it had been much bigger the moon would have had a core of ice. · Powered by Phabricator