Skip to content

Bookinfo latest#19

Merged
rshriram merged 10 commits intomasterfrom
bookinfo_latest
Feb 27, 2017
Merged

Bookinfo latest#19
rshriram merged 10 commits intomasterfrom
bookinfo_latest

Conversation

@frankbu
Copy link
Copy Markdown
Contributor

@frankbu frankbu commented Feb 27, 2017

Demo improvements and delay injection fixed.


This change is Reviewable

@googlebot
Copy link
Copy Markdown
Collaborator

So there's good news and bad news.

👍 The good news is that everyone that needs to sign a CLA (the pull request submitter and all commit authors) have done so. Everything is all good there.

😕 The bad news is that it appears that one or more commits were authored by someone other than the pull request submitter. We need to confirm that they're okay with their commits being contributed to this project. Please have them confirm that here in the pull request.

Note to project maintainer: This is a terminal state, meaning the cla/google commit status will not change from this state. It's up to you to confirm consent of the commit author(s) and merge this pull request when appropriate.

@googlebot googlebot added the cla: no Set by the Google CLA bot to indicate the author of a PR has not signed the Google CLA. label Feb 27, 2017
@frankbu frankbu requested a review from rshriram February 27, 2017 17:16
@rshriram rshriram merged commit 0846a2a into master Feb 27, 2017
rshriram added a commit that referenced this pull request Oct 30, 2017
Bookinfo latest

Former-commit-id: 0846a2a
mandarjog pushed a commit that referenced this pull request Oct 31, 2017
* Wrap extra long lines
* Add/Fix comments for public methods and structs
* Always check the error returned by a method
* Use narrow interfaces

This closes #19 and is a precommit for #22 .
mandarjog pushed a commit that referenced this pull request Oct 31, 2017
* add crd files

* change to example directory

* modify yaml.

* sync with api

* update
mandarjog pushed a commit that referenced this pull request Oct 31, 2017
* Cache consistency tests and validation

* Correct a comment

* Fix a test
mandarjog pushed a commit that referenced this pull request Nov 2, 2017
* Wrap extra long lines
* Add/Fix comments for public methods and structs
* Always check the error returned by a method
* Use narrow interfaces

This closes #19 and is a precommit for #22 .
mandarjog pushed a commit that referenced this pull request Nov 2, 2017
Bookinfo latest

Former-commit-id: 0846a2a
kyessenov pushed a commit to kyessenov/istio that referenced this pull request Aug 13, 2018
rajusharma pushed a commit to rajusharma/istio that referenced this pull request Jul 2, 2019
Move MCP-address to mesh config (instead of using flag).
vikaschoudhary16 pushed a commit to vikaschoudhary16/istio that referenced this pull request Oct 11, 2019
These changes intend to sync oap helm yamls with the latest skywalking.

 * Using `subPath` to override some parts of config files.
 * Parameterize `resources` for CI to use a lower memory.
 * moving `official_analysis .oal` from jar to config folder. if UI or user need a new metric, we don’t build a new spm image.
howardjohn referenced this pull request in howardjohn/istio Jan 12, 2020
howardjohn pushed a commit that referenced this pull request Mar 25, 2020
danehans pushed a commit to danehans/istio that referenced this pull request Nov 2, 2021
@LugiaChang LugiaChang mentioned this pull request Sep 19, 2022
16 tasks
antonioberben pushed a commit to antonioberben/istio that referenced this pull request Jan 29, 2024
@pavan359t pavan359t mentioned this pull request Sep 6, 2024
2 tasks
luksa pushed a commit to luksa/istio that referenced this pull request Oct 14, 2024
…ster-merge_upstream_istio_master-6253864e

Automator: merge upstream changes to openshift-service-mesh/istio@master
johnlanni added a commit to johnlanni/istio that referenced this pull request Jan 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: no Set by the Google CLA bot to indicate the author of a PR has not signed the Google CLA.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants