-
Notifications
You must be signed in to change notification settings - Fork 12
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
Create a test scenario for mixed version clustering #720
base: main
Are you sure you want to change the base?
Conversation
if not config.SNAP: | ||
pytest.fail("Set TEST_SNAP to the path where the snap is") | ||
|
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.
moved to test_util.util
snap_path = (tmp_path / "k8s.snap").as_posix() | ||
|
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.
this is gone because we can install from all kinds of things now...track, channel, revision, or file
75d5fd7
to
41fe426
Compare
41fe426
to
d8aa2bc
Compare
50b41c7
to
5b3b8b1
Compare
e8eaf59
to
a88f1ce
Compare
@@ -12,40 +12,45 @@ | |||
|
|||
@pytest.mark.node_count(1) | |||
@pytest.mark.no_setup() | |||
@pytest.mark.xfail(reason="cilium failures are blocking this from working") |
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.
@bschimke95 I did not see this to be the case any longer. I've fixed this test again and it seems to pass??
Overview
node_count
ornode_config
when you wish to configure the nodes differently (say with a different snap version)Details