Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fixes #3200 #3201

Closed
wants to merge 1 commit into from
Closed

fixes #3200 #3201

wants to merge 1 commit into from

Conversation

iocampomx
Copy link

@iocampomx iocampomx commented Aug 15, 2017

After include this two policy statements related to AWS IAM nodes role, I was able to run the cluster successfully.

@k8s-ci-robot
Copy link
Contributor

Thanks for your pull request. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

📝 Please follow instructions at https://github.com/kubernetes/kubernetes/wiki/CLA-FAQ to sign the CLA.

It may take a couple minutes for the CLA signature to be fully registered; after that, please reply here with a new comment and we'll verify. Thanks.


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.

@k8s-ci-robot k8s-ci-robot added the cncf-cla: no Indicates the PR's author has not signed the CNCF CLA. label Aug 15, 2017
@k8s-ci-robot
Copy link
Contributor

Hi @Nafiux. Thanks for your PR.

I'm waiting for a kubernetes 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.

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.

@k8s-ci-robot k8s-ci-robot added the needs-ok-to-test Indicates a PR that requires an org member to verify it is safe to test. label Aug 15, 2017
@k8s-github-robot
Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: nafiux
We suggest the following additional approver: zmerlynn

Assign the PR to them by writing /assign @zmerlynn in a comment when ready.

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

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

@iocampomx
Copy link
Author

I just signed the CLA

@StevenACoffman
Copy link
Contributor

StevenACoffman commented Aug 15, 2017

This PR is for #3200 and should be retitled
IAM role statement for nodes is missing: arn:aws:s3:::bucket/dev.bucket/pki/private/ca/*

@gambol99
Copy link
Contributor

I've updated the initial issue #3200 ... i'm not sure this is a bug, is stems from the fact they are using a cli built off master branch but most likely using the nodeup from 1.7.0 tag.

@iocampomx
Copy link
Author

I've updated the initial issue #3200, it seems that this problem is caused because brew install kops is pointing to a different version.

@iocampomx iocampomx closed this Aug 16, 2017
@gambol99
Copy link
Contributor

note, we added #3210 as it can effect people inadvertently building off master

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cncf-cla: no Indicates the PR's author has not signed the CNCF CLA. needs-ok-to-test Indicates a PR that requires an org member to verify it is safe to test.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants