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

Update CI jobs to not require contexts for PR builds #3219

Merged
merged 1 commit into from
May 20, 2021
Merged

Update CI jobs to not require contexts for PR builds #3219

merged 1 commit into from
May 20, 2021

Conversation

owais
Copy link
Contributor

@owais owais commented May 18, 2021

This PR contains the same fix to unblock community PRs as the following contrib PR: open-telemetry/opentelemetry-collector-contrib#3423

@gramidt
Copy link
Member

gramidt commented May 19, 2021

@owais - Are you able to add in the changes from open-telemetry/opentelemetry-collector-contrib#3423 to the core collector?

cc @tigrannajaryan

@owais owais changed the title test protected jobs Update CI jobs to not require contexts for PR builds May 20, 2021
@@ -14,5 +14,5 @@ else
curr="${TESTS[$i]}"
fi
done
MATRIX+="]}"
MATRIX+=",{\"test\":\"$curr\"}]}"
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This script actually misses the last two load tests. This change ensures last two load tests are added to the matrix.

@owais owais marked this pull request as ready for review May 20, 2021 02:12
@owais owais requested a review from a team May 20, 2021 02:12
This commit ensures that PR builds will not use circleci contexts as any
jobs depending on contexts can only be executed by otel members.
Copy link
Member

@tigrannajaryan tigrannajaryan left a comment

Choose a reason for hiding this comment

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

Thanks @owais

@tigrannajaryan tigrannajaryan merged commit d3c1a58 into open-telemetry:main May 20, 2021
@owais owais deleted the skip-protected-jobs-from-forked-builds branch May 20, 2021 13:40
dashpole pushed a commit to dashpole/opentelemetry-collector that referenced this pull request Jun 14, 2021
This commit ensures that PR builds will not use circleci contexts as any
jobs depending on contexts can only be executed by otel members.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants