Skip to content

Fix nil stackdriver environment variables#18710

Merged
istio-testing merged 1 commit intoistio:masterfrom
gargnupur:nup_tracing_fix
Nov 7, 2019
Merged

Fix nil stackdriver environment variables#18710
istio-testing merged 1 commit intoistio:masterfrom
gargnupur:nup_tracing_fix

Conversation

@gargnupur
Copy link
Copy Markdown
Contributor

@gargnupur gargnupur commented Nov 6, 2019

Fix nil stackdriver environment variables.
Ref: #18680

@gargnupur gargnupur requested a review from a team as a code owner November 6, 2019 21:16
@googlebot googlebot added the cla: yes Set by the Google CLA bot to indicate the author of a PR has signed the Google CLA. label Nov 6, 2019
@istio-testing istio-testing added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Nov 6, 2019
Copy link
Copy Markdown
Member

@howardjohn howardjohn left a comment

Choose a reason for hiding this comment

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

Need port to istio/installer

@gargnupur
Copy link
Copy Markdown
Contributor Author

Need port to istio/installer

@howardjohn : where do we need to make changes for that?

@howardjohn
Copy link
Copy Markdown
Member

/retest

1 similar comment
@howardjohn
Copy link
Copy Markdown
Member

/retest

@howardjohn
Copy link
Copy Markdown
Member

howardjohn commented Nov 7, 2019 via email

@istio-testing istio-testing merged commit 45908a9 into istio:master Nov 7, 2019
@gargnupur
Copy link
Copy Markdown
Contributor Author

@mandarjog : cherrypick this to 1.4?

@gargnupur
Copy link
Copy Markdown
Contributor Author

/cherry-pick release-1.4

@istio-testing
Copy link
Copy Markdown
Collaborator

@gargnupur: new pull request created: #18812

Details

In response to this:

/cherry-pick release-1.4

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

sdake pushed a commit to sdake/istio that referenced this pull request Dec 1, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/extensions and telemetry cla: yes Set by the Google CLA bot to indicate the author of a PR has signed the Google CLA. size/S Denotes a PR that changes 10-29 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants