Skip to content

Split the feature_flags_SUITE into two parts for CI/Bazel #3443

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

Merged
merged 1 commit into from
Sep 17, 2021

Conversation

HoloRin
Copy link
Contributor

@HoloRin HoloRin commented Sep 17, 2021

Two testcases in the original suite fail if the test is run as the root user. Currently under remote execution with bazel this is the only working option. There is a workaround in place (executing the feature_flags_SUITE on the Github Actions worker rather than on BuildBuddy), but the entire suite when run that way takes around 12 minutes. This splits the suite so that the minimal set of cases is executed using the slower workaround.

Two testcases in the original suite fail if the test is run as the
root user. Currently under remote execution with bazel this is the
only working option. There is a workaround in place, but the entire
suite when run that way takes around 12 minutes. This splits the suite
so that the minimal set of cases is executed using the slower workaround.
@mergify mergify bot added the bazel label Sep 17, 2021
@HoloRin HoloRin merged commit bc05489 into master Sep 17, 2021
@HoloRin HoloRin deleted the split-feature-flags-suite-for-bazel branch September 17, 2021 09:21
HoloRin added a commit that referenced this pull request Sep 17, 2021
Split the feature_flags_SUITE into two parts for CI/Bazel (backport #3443)
HoloRin added a commit that referenced this pull request Sep 17, 2021
Split the feature_flags_SUITE into two parts for CI/Bazel (backport #3443) (backport #3444)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant