Skip to content
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

ci/e2e: add a sequential option to boostrap nodes #724

Merged
merged 3 commits into from
Mar 10, 2023

Conversation

ldevulder
Copy link
Contributor

@ldevulder ldevulder commented Mar 10, 2023

We have to cover most cases: parallel bootstrapping as well as sequential ones, as we can have different behaviors and so different issues.

Verification runs:

@ldevulder ldevulder self-assigned this Mar 10, 2023
@ldevulder ldevulder marked this pull request as draft March 10, 2023 11:05
@ldevulder ldevulder force-pushed the ci-add-sequential-test branch 2 times, most recently from e2c3d0d to 06898b9 Compare March 10, 2023 11:27
We have to cover most cases: parallel bootstrapping as well as
sequential ones, as we can have different behaviors and so different
issues.

Signed-off-by: Loic Devulder <ldevulder@suse.com>
@ldevulder ldevulder marked this pull request as ready for review March 10, 2023 14:15
Copy link
Contributor

@juadk juadk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, just a small typo.

.github/workflows/cli-obs-manual-workflow.yaml Outdated Show resolved Hide resolved
.github/workflows/master-e2e.yaml Outdated Show resolved Hide resolved
ldevulder and others added 2 commits March 10, 2023 15:29
Co-authored-by: Julien ADAMEK <julien.adamek@suse.com>
Signed-off-by: Loic Devulder <ldevulder@suse.com>
Co-authored-by: Julien ADAMEK <julien.adamek@suse.com>
Signed-off-by: Loic Devulder <ldevulder@suse.com>
@ldevulder ldevulder merged commit 5d24b1f into main Mar 10, 2023
@ldevulder ldevulder deleted the ci-add-sequential-test branch March 10, 2023 15:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants