-
Notifications
You must be signed in to change notification settings - Fork 41
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
integration tests: invalid bluechi agent port #482
Conversation
I will send more patches to complete #416. One test per file to simplify the code/review/test. We should cover all agent/controller options. |
Funny to see that even with invalid configuration GitHub Actions is "able to start" (result 0): From the failure logs:
|
Testing locally the new validation: OS:
Running with tmt:
|
changing the approach to using the output string from
|
e9026a5
to
10772b1
Compare
okay, found that during the tmt tests in Github Actions systemd was trying to restart the service and the output of systemctl status bluechi-agent was "activating" but we had a failure count = 1. Updated the test, green light to review and merge if accepted. Locally also pass:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A few NITs, but otherwise looks great!
...ests/tier0/bluechi-agent-invalid-port-configuration/test_agent_invalid_port_configuration.py
Outdated
Show resolved
Hide resolved
...ests/tier0/bluechi-agent-invalid-port-configuration/test_agent_invalid_port_configuration.py
Outdated
Show resolved
Hide resolved
...ests/tier0/bluechi-agent-invalid-port-configuration/test_agent_invalid_port_configuration.py
Outdated
Show resolved
Hide resolved
This test includes three nodes: bluechi-controller, node-foo, node-bar. - bluechi-controller is the controller node - node-foo will start the agent service with normal configuration using port 8420 - node-bar will try to start the agent service with incorrect port (842O) and should fail (as expected) Signed-off-by: Douglas Schilling Landgraf <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Thanks for the comments, let's make it fly! One more test integration added \o/ |
This test includes three nodes: bluechi-controller, node-foo, node-bar.
Related: #416
Signed-off-by: Douglas Schilling Landgraf [email protected]