Skip to content

Conversation

wattli
Copy link
Contributor

@wattli wattli commented May 30, 2018

No description provided.

Jianfei Hu and others added 6 commits May 29, 2018 12:47
* Move the mesh expansion node agent into the pkg/nodeagent.
* Add a gcp subpackage for nodeagent.
* Lint formatting.
…ly (istio#5838)

* Add documents and scripts for k8s+vagrant env.

* update macOS setup

* Update and rename setup_linux_prereqs.sh to linux_prereqs.sh

* Update localregistry.yaml

* Update README.md

* Update README.md

* Update README.md

* Update README.md

* Update README.md

* Update README.md

* Update README.md

* Update README.md

* Update README.md

* Update README.md

* Update startup.sh

* Update README.md

* update filename and readme

* update readme

* update image

* Create prereqs.sh

* Update README.md

* update README

* update readme for cleanup

* add insecure registry setting onto vm

* Add dlv debugging installation to vm setup

* Change vm sync folder

* Update setup_kubectl_host.sh

* Update vm_setup.sh

* Create OWNERS (istio#5130)

* Update vm_setup.sh (istio#5135)

* Update Vagrantfile (istio#5134)

* Update README.md (istio#5138)

* Update README.md (istio#5141)

* Update README.md (istio#5142)

* Update README.md (istio#5154)

* Update README.md

* Update README.md

* Update README.md

* Update README.md

* Add localregistry.yaml specific for vagrant (istio#5181)

* Create localregistry.yaml

* Revert back changes in localregistry.yaml

* Updated VagrantFile

Updated VagrantFile as per new location of localregistry.yaml

* Make test available on macOS  (istio#5177)

* update test setup for macos

* add local test flag

* modify init.sh

* update docker check

* update env setting

* update USE_DOCKER

* export GOOS=linux to docker and push targets

* Move files out of RunTestOnHost and remove RunTestOnHost and RunTestOnVm. (istio#5206)

* Move files.

* Revise per review comments.

* Revise per comment.

* Polish and simplify README.md (istio#5211)

* Polish and simplify README.md

* Update README.md

* Update README.md

* Rename scripts and cleanup unused scripts. (istio#5215)

* Move files.

* Rename scripts and remove unused scripts.

* Update README.md

* Remove unnecessary env setting (istio#5217)

* remove space at the end of export

* modify env changes

* Update prerequisites and setup scripts (istio#5220)

* remove space at the end of export

* modify env changes

* modify setups

* update prereqs scripts for macos

* Update README.md

* Cleanup test_setup.sh and setup_kubelet_config_host.sh (istio#5230)

* Protect Copy to ~/.kube/config_old

* Update setup_test.sh

* Update setup_dockerdaemon_linux.sh

* Push data from help commands to > /dev/null

Push data from help commands to > /dev/null as otherwise it fills up screen with too much info and we won't come to know about failures easily

* Update setup_test.sh

* Update README.md

* Update setup_dockerdaemon_linux.sh

* Update setup_kubelet_config_host.sh

* Create Troubleshooting.md (istio#5235)

* Clean up and update scripts (istio#5233)

* remove space at the end of export

* modify env changes

* modify setups

* update prereqs scripts for macos

* modify and cleanup the scripts

* rename files

* cleanup

* update scripts

* changes update

* update

* update Vagrantfile

* update local registry setting

* add comment back

* Update Vagrantfile (istio#5239)

* Update Troubleshooting.md (istio#5237)

* Update Troubleshooting.md

* Update Troubleshooting.md

* Update Troubleshooting.md

* Update Troubleshooting.md

* Update Troubleshooting.md

* Update Troubleshooting.md

* Small changes to scripts from feedback (istio#5449)

* remove space at the end of export

* modify env changes

* modify setups

* update prereqs scripts for macos

* modify and cleanup the scripts

* rename files

* cleanup

* update scripts

* changes update

* update

* update Vagrantfile

* update local registry setting

* add comment back

* Add changes to setup scripts

* update kubectl setup script

* Separate debian linux distribution (istio#5515)

* Separate debian linux distribution

* add default case

* Fix download path for kubectl on mac (istio#5553)

* Update Troubleshooting.md (istio#5240)

* Update Troubleshooting.md

* Update Troubleshooting.md

* Fix minor issues with scripts (istio#5555)

* Update install_prereqs_macos.sh

* Update install_prereqs_debian.sh

* Update install_prereqs_debian.sh

* Update Troubleshooting.md

* Update Troubleshooting.md

* Update Troubleshooting.md

* Update README.md

* Update Troubleshooting.md

* Update Troubleshooting.md

* Update Troubleshooting.md

* Adding version of virtualbox installed

* Setup local testing using minikube

* Update install_prereqs_debian.sh

* Format script

* Update cleanup_host.sh

* Update cleanup_host.sh

* Update setup_host.sh

* Revise per comment.

* Update go installation.

* Revise per comment.
Signed-off-by: Kuat Yessenov <kuat@google.com>
@googlebot
Copy link
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 or co-authored by someone other than the pull request submitter. We need to confirm that all authors are ok 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.

@istio-testing
Copy link
Collaborator

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: wattli

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

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@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 May 30, 2018
@wattli wattli merged commit b31e38b into istio:lita-test May 30, 2018
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.

6 participants