Skip to content

Conversation

myidpt
Copy link

@myidpt myidpt commented Oct 31, 2017

No description provided.

@istio-testing
Copy link
Contributor

Jenkins job istio.github.io/presubmit passed


The following command downloads the example files:
```bash
wget https://raw.githubusercontent.com/istio/auth/master/samples/plugin_ca_certs/ca-cert.pem
Copy link
Member

Choose a reason for hiding this comment

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

change to new location

Copy link
Author

Choose a reason for hiding this comment

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

Done.


## Further reading

* Read the Istio [CA arguments](https://github.com/istio/auth/blob/master/cmd/istio_ca/main.go).
Copy link
Member

Choose a reason for hiding this comment

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

change to new location

Copy link
Author

Choose a reason for hiding this comment

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

Done.

Copy link
Member

@ldemailly ldemailly left a comment

Choose a reason for hiding this comment

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

thx

@myidpt
Copy link
Author

myidpt commented Nov 6, 2017

/cc @wattli

@ldemailly
Copy link
Member

/retest

@myidpt
Copy link
Author

myidpt commented Nov 8, 2017

@geeknoid Hi Martin, Seb and Yutong suggested that you are managing this repo. So do you know who should I contact to solve the presubmit error, or can we just ignore the error? It would be great if this PR can be merged this week. Thanks!

@ldemailly
Copy link
Member

you need to fix the error:

@myidpt
Copy link
Author

myidpt commented Nov 8, 2017

@ldemailly Fixing the error in #707

@istio-testing
Copy link
Contributor

Jenkins job istio.github.io/presubmit passed

@ldemailly ldemailly merged commit 4a29314 into istio:master Nov 8, 2017
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.

3 participants