Skip to content

Commit 395b644

Browse files
author
Arvind Jangir
committed
fix tests and enable bmc in CI
1 parent 73f72c9 commit 395b644

2 files changed

Lines changed: 3 additions & 3 deletions

File tree

.github/workflows/test.yml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -138,6 +138,7 @@ jobs:
138138
--add-feature azure-rm \
139139
--add-feature google \
140140
--add-feature remote-execution \
141+
--add-feature bmc \
141142
${{ matrix.iop == 'enabled' && '--add-feature iop' || '' }}
142143
- name: Run tests
143144
run: |

tests/foreman_proxy_test.py

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -5,13 +5,12 @@
55
from conftest import enabled_features
66

77
FOREMAN_PROXY_PORT = 8443
8-
BMC_CONFIG = "/etc/foreman-proxy/settings.d/bmc.yml"
98

10-
def get_proxy_feature_setting(server, feature, default=None):
9+
def get_proxy_feature_setting(server, feature):
1110
cmd = server.run(f"podman exec foreman-proxy cat /etc/foreman-proxy/settings.d/{feature}.yml")
1211
if cmd.succeeded:
1312
return yaml.safe_load(cmd.stdout)
14-
return default
13+
return {}
1514

1615
def is_bmc_enabled():
1716
return 'bmc' in enabled_features()

0 commit comments

Comments
 (0)