Skip to content

Clarify environment variables set in .profile#919

Merged
istio-merge-robot merged 1 commit intoistio:masterfrom
sdake:918
Sep 25, 2017
Merged

Clarify environment variables set in .profile#919
istio-merge-robot merged 1 commit intoistio:masterfrom
sdake:918

Conversation

@sdake
Copy link
Copy Markdown
Member

@sdake sdake commented Sep 23, 2017

Fixes (#918)

Prior to this small change, the .profile settings were not completely
obvious. Now all profile settings that Istio should expect to use
are set in one place.

Release note:

NONE

@istio-testing
Copy link
Copy Markdown
Collaborator

Hi @sdake. Thanks for your PR.

I'm waiting for a istio member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.

I understand the commands that are listed here.

Details

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. I understand the commands that are listed here.

@sdake
Copy link
Copy Markdown
Member Author

sdake commented Sep 23, 2017

/assign @andraxylia

Fixes (#918)

Prior to this small change, the .profile settings were not completely
obvious.  Now all profile settings that Istio should expect to use
are set in one place.
Copy link
Copy Markdown
Member

@gyliu513 gyliu513 left a comment

Choose a reason for hiding this comment

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

/lgtm

@gyliu513
Copy link
Copy Markdown
Member

@sdake glad to see you again in istio!

@sdake
Copy link
Copy Markdown
Member Author

sdake commented Sep 24, 2017

@gyliu513 yo dude, morning, Istio lucky to have you 👍

Copy link
Copy Markdown
Member

@rshriram rshriram left a comment

Choose a reason for hiding this comment

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

/lgtm

@rshriram
Copy link
Copy Markdown
Member

/ok-to-test

@sdake
Copy link
Copy Markdown
Member Author

sdake commented Sep 25, 2017

/test e2e-suite-rbac-no_auth

@sdake
Copy link
Copy Markdown
Member Author

sdake commented Sep 25, 2017

/release-note-none

@rshriram
Copy link
Copy Markdown
Member

/lgtm

@istio-merge-robot
Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: gyliu513, rshriram

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these OWNERS Files:

You can indicate your approval by writing /approve in a comment
You can cancel your approval by writing /approve cancel in a comment

@istio-merge-robot
Copy link
Copy Markdown

/test all [submit-queue is verifying that this PR is safe to merge]

@istio-merge-robot
Copy link
Copy Markdown

Automatic merge from submit-queue

@istio-merge-robot istio-merge-robot merged commit 977a1af into istio:master Sep 25, 2017
istio-merge-robot pushed a commit that referenced this pull request Sep 27, 2017
Automatic merge from submit-queue

Clarify how GITHUB_USER should be setup in the development guide.

Fixes (#918)

This is a follow up change to PR #919: Clarify environment variables set in
.profile

Prior to this small change, the .profile settings were not completely obvious.
Now all profile settings that Istio should expect to use are set in one place

**Release Note**

NONE

**Release note**:

```release-note
NONE
```
rshriram pushed a commit that referenced this pull request Oct 30, 2017
Automatic merge from submit-queue

Clarify environment variables set in .profile

Fixes (#918)

Prior to this small change, the .profile settings were not completely
obvious.  Now all profile settings that Istio should expect to use
are set in one place.

**Release note**:

NONE

Former-commit-id: 977a1af
rshriram pushed a commit that referenced this pull request Oct 30, 2017
Automatic merge from submit-queue

Clarify how GITHUB_USER should be setup in the development guide.

Fixes (#918)

This is a follow up change to PR #919: Clarify environment variables set in
.profile

Prior to this small change, the .profile settings were not completely obvious.
Now all profile settings that Istio should expect to use are set in one place

**Release Note**

NONE

**Release note**:

```release-note
NONE
```

Former-commit-id: 8bb136a
vbatts pushed a commit to vbatts/istio that referenced this pull request Oct 31, 2017
Automatic merge from submit-queue

Clarify environment variables set in .profile

Fixes (istio#918)

Prior to this small change, the .profile settings were not completely
obvious.  Now all profile settings that Istio should expect to use
are set in one place.

**Release note**:

NONE

Former-commit-id: 977a1af
vbatts pushed a commit to vbatts/istio that referenced this pull request Oct 31, 2017
…tio#947)

Automatic merge from submit-queue

Clarify how GITHUB_USER should be setup in the development guide.

Fixes (istio#918)

This is a follow up change to PR istio#919: Clarify environment variables set in
.profile

Prior to this small change, the .profile settings were not completely obvious.
Now all profile settings that Istio should expect to use are set in one place

**Release Note**

NONE

**Release note**:

```release-note
NONE
```

Former-commit-id: 8bb136a
mandarjog pushed a commit that referenced this pull request Nov 2, 2017
Automatic merge from submit-queue

Clarify environment variables set in .profile

Fixes (#918)

Prior to this small change, the .profile settings were not completely
obvious.  Now all profile settings that Istio should expect to use
are set in one place.

**Release note**:

NONE

Former-commit-id: 977a1af
mandarjog pushed a commit that referenced this pull request Nov 2, 2017
Automatic merge from submit-queue

Clarify how GITHUB_USER should be setup in the development guide.

Fixes (#918)

This is a follow up change to PR #919: Clarify environment variables set in
.profile

Prior to this small change, the .profile settings were not completely obvious.
Now all profile settings that Istio should expect to use are set in one place

**Release Note**

NONE

**Release note**:

```release-note
NONE
```

Former-commit-id: 8bb136a
kyessenov pushed a commit to kyessenov/istio that referenced this pull request Aug 13, 2018
* Use mixerclient with status code fix

* Update config.yml
0x01001011 pushed a commit to thedemodrive/istio that referenced this pull request Jul 16, 2020
luksa pushed a commit to luksa/istio that referenced this pull request Apr 11, 2024
Signed-off-by: Jacek Ewertowski <jewertow@redhat.com>
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.

7 participants