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.

Sign up for GitHub

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

Jump to bottom

Update builder integration tests, enable daily/manual builds #4771

Merged
merged 3 commits into from
Feb 1, 2022
Merged

Update builder integration tests, enable daily/manual builds #4771

merged 3 commits into from
Feb 1, 2022

Conversation

jpkrohling
Copy link
Member

This PR bumps the integration tests to use the latest released version of the collector when building a distribution. It also enables a daily build running the integration tests, as well as their manual execution.

Signed-off-by: Juraci Paixão Kröhling [email protected]

@jpkrohling jpkrohling requested review from a team and bogdandrutu January 31, 2022 19:16
@jpkrohling jpkrohling mentioned this pull request Jan 31, 2022
Merged
@codecov
Copy link

codecov bot commented Jan 31, 2022 •
edited
Loading

Codecov Report

Merging #4771 (9f94107) into main (fde1076) will increase coverage by 0.00%.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #4771   +/-   ##
=======================================
  Coverage   90.70%   90.70%           
=======================================
  Files         179      179           
  Lines       10565    10569    +4     
=======================================
+ Hits         9583     9587    +4     
  Misses        763      763           
  Partials      219      219
Impacted Files Coverage Δ
service/config_provider.go 92.24% <0.00%> (+0.27%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update fde1076...9f94107. Read the comment docs.

@jpkrohling jpkrohling added the Skip Changelog PRs that do not require a CHANGELOG.md entry label Jan 31, 2022
@bogdandrutu bogdandrutu merged commit a1322ae into open-telemetry:main Feb 1, 2022
bogdandrutu pushed a commit to bogdandrutu/opentelemetry-collector that referenced this pull request Feb 1, 2022
@jpkrohling @bogdandrutu
Update builder integration tests, enable daily/manual builds (open-te…
56c49fb 
…lemetry#4771)

* Update builder integration tests, enable daily/manual builds

Signed-off-by: Juraci Paixão Kröhling <[email protected]>

* Update core.builder.yaml

* Update replaces.builder.yaml

Co-authored-by: Bogdan Drutu <[email protected]>
bogdandrutu pushed a commit to bogdandrutu/opentelemetry-collector that referenced this pull request Feb 1, 2022
@jpkrohling @bogdandrutu
Update builder integration tests, enable daily/manual builds (open-te…
4e8657f 
…lemetry#4771)

* Update builder integration tests, enable daily/manual builds

Signed-off-by: Juraci Paixão Kröhling <[email protected]>

* Update core.builder.yaml

* Update replaces.builder.yaml

Co-authored-by: Bogdan Drutu <[email protected]>
bogdandrutu pushed a commit to bogdandrutu/opentelemetry-collector that referenced this pull request Feb 1, 2022
@jpkrohling @bogdandrutu
Update builder integration tests, enable daily/manual builds (open-te…
52ccb18 
…lemetry#4771)

* Update builder integration tests, enable daily/manual builds

Signed-off-by: Juraci Paixão Kröhling <[email protected]>

* Update core.builder.yaml

* Update replaces.builder.yaml

Co-authored-by: Bogdan Drutu <[email protected]>
bogdandrutu pushed a commit to bogdandrutu/opentelemetry-collector that referenced this pull request Mar 21, 2022
@jpkrohling @bogdandrutu
Update builder integration tests, enable daily/manual builds (open-te…
d39305e 
…lemetry#4771)

* Update builder integration tests, enable daily/manual builds

Signed-off-by: Juraci Paixão Kröhling <[email protected]>

* Update core.builder.yaml

* Update replaces.builder.yaml

Co-authored-by: Bogdan Drutu <[email protected]>
bogdandrutu pushed a commit to bogdandrutu/opentelemetry-collector that referenced this pull request Mar 21, 2022
@jpkrohling @bogdandrutu
Update builder integration tests, enable daily/manual builds (open-te…
4279037 
…lemetry#4771)

* Update builder integration tests, enable daily/manual builds

Signed-off-by: Juraci Paixão Kröhling <[email protected]>

* Update core.builder.yaml

* Update replaces.builder.yaml

Co-authored-by: Bogdan Drutu <[email protected]>
bogdandrutu added a commit that referenced this pull request Mar 22, 2022
@bogdandrutu @jpkrohling
Update builder integration tests, enable daily/manual builds (#4771) (#…
4c3b824 
…4783)

* Update builder integration tests, enable daily/manual builds

Signed-off-by: Juraci Paixão Kröhling <[email protected]>

* Update core.builder.yaml

* Update replaces.builder.yaml

Co-authored-by: Bogdan Drutu <[email protected]>

Co-authored-by: Juraci Paixão Kröhling <[email protected]>
Nicholaswang pushed a commit to Nicholaswang/opentelemetry-collector that referenced this pull request Jun 7, 2022
@bogdandrutu @jpkrohling @Nicholaswang
Update builder integration tests, enable daily/manual builds (open-te…
1e1257b 
…lemetry#4771) (open-telemetry#4783)

* Update builder integration tests, enable daily/manual builds

Signed-off-by: Juraci Paixão Kröhling <[email protected]>

* Update core.builder.yaml

* Update replaces.builder.yaml

Co-authored-by: Bogdan Drutu <[email protected]>

Co-authored-by: Juraci Paixão Kröhling <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@bogdandrutu bogdandrutu bogdandrutu approved these changes

Assignees
No one assigned
Labels
Skip Changelog PRs that do not require a CHANGELOG.md entry
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@jpkrohling @bogdandrutu