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

Add OAP+Istio(mixer) to CI pipeline #3904

Merged
merged 81 commits into from Dec 3, 2019
Merged

Add OAP+Istio(mixer) to CI pipeline #3904

merged 81 commits into from Dec 3, 2019

Conversation

innerpeacez
Copy link
Member

@innerpeacez innerpeacez commented Nov 20, 2019

Please answer these questions before submitting pull request

  • Why submit this pull request?
  • Bug fix
  • New feature provided
  • Improve performance

for #3771

@wu-sheng wu-sheng added this to the 6.6.0 milestone Nov 20, 2019
@wu-sheng wu-sheng added the CI settings CI settings adjustment label Nov 20, 2019
@wu-sheng
Copy link
Member

ES7 storage has been merged and updates to your branch.

@codecov-io
Copy link

Codecov Report

Merging #3904 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@          Coverage Diff           @@
##           master   #3904   +/-   ##
======================================
  Coverage    27.9%   27.9%           
======================================
  Files        1122    1122           
  Lines       24341   24341           
  Branches     3528    3528           
======================================
  Hits         6792    6792           
  Misses      16949   16949           
  Partials      600     600

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 bdc0609...4fb1768. Read the comment docs.

@wu-sheng
Copy link
Member

wu-sheng commented Dec 2, 2019

Please resolve the conflict. Should we merge this first, then working on verification in a new PR? This seems a little too long commit logs. As title Add OAP+Istio(mixer) to CI pipeline, this should be OK for now.

@innerpeacez @hanahmily Thoughts? @hanahmily Please review.

@innerpeacez
Copy link
Member Author

@wu-sheng conflict fixed.I think it can be merged, I will create a new pr to complete the data verification steps.

@wu-sheng wu-sheng changed the title [WIP] Add OAP+Istio(mixer) to CI pipeline Add OAP+Istio(mixer) to CI pipeline Dec 3, 2019
@wu-sheng wu-sheng merged commit 3e22ff5 into apache:master Dec 3, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI settings CI settings adjustment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants