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

[Machine] - Amazon - When machine creation fails because of invalid access key , transition message does not include the full error message received from amazon. #372

Closed
sangeethah opened this issue Mar 31, 2015 · 1 comment
Labels
kind/bug Issues that are defects reported by users or that we know have reached a real release

Comments

@sangeethah
Copy link
Contributor

Server Version - V0.14.0

Steps to reproduce the problem:

Scenario 1:

Try to add a Amazon host by providing invalid access key.

Machine creation fails with the state being set to "error", but the transition error message does not contain the full error message from amazon:

Machine detail from API:
"data": {
"fields": {
"virtualboxConfig": null,
"amazonec2Config": {
"vpcId": "vpc-44144621",
"accessKey": "***_",
"secretKey": "_
",
"securityGroup": "testsangee",
},
"digitaloceanConfig": null,
"driver": "amazonec2",
"transitioningMessage": "exit status 1",
"transitioning": "error",
},
},

time="2015-03-30T23:45:25Z" level=info msg="stdout: time="2015-03-30T23:45:25Z" level="error" msg="Error creating machine: Problem with AWS API call: Non-200 API response: code=401 message=AWS was not able to validate the provided access credentials\n" " resourceId: =1ph15

Scenario 2:

Try to add a Amazon host by providing invalid secret key.

Machine creation fails with the state being set to "error", but the transition error message does not contain the full error message from amazon .

Error message presented to user - "Error creating machine: Problem with AWS API call: Non-200 API response: code"

API:

"data": {
"fields": {
"amazonec2Config": {
"region": "us-east-1",
"instanceType": "t2.micro",
"zone": "a",
"rootSize": "16",
"accessKey": "*********_",
"vpcId": "vpc-44144621",
"secretKey": "G
_****",
"ami": "",
"iamInstanceProfile": "",
},
"driver": "amazonec2",
"transitioningMessage": "Error creating machine: Problem with AWS API call: Non-200 API response: code",
"transitioning": "error",
},
},
Server logs having the full error message:

Server logs having the full error message:

time="2015-03-31T01:24:10Z" level=info msg="stdout: time="2015-03-31T01:24:10Z" level="error" msg="Error creating machine: Problem with AWS API call: Non-200 API response: code=401 message=AWS was not able to validate the provided access credentials\n" " resourceId: =1ph21

@sangeethah sangeethah added the kind/bug Issues that are defects reported by users or that we know have reached a real release label Mar 31, 2015
@deniseschannon
Copy link

Verified Fix in v0.14.2

There was 1 case where I got 'exit status 1' instead of the true error message. But I have been unable to reproduce. I will reopen if I see it again.

pmatseykanets pushed a commit to pmatseykanets/rancher that referenced this issue Feb 5, 2024
…#372)

* Add a check for specific fields we don't want in the headers
pmatseykanets added a commit that referenced this issue Feb 5, 2024
* Add a check for specific fields we don't want in the headers (#372)
* Split reconcileProjectAccessToGlobalResources
* Add tests
* Check APIGroup
* Add tests and change cr client used
* [2.8] Bump API-UI version #432
* Update norman and apiserver
* Update RKE to 1.5.3
* Regenerate files after updating RKE to 1.5.3
* Update runc to 1.1.12

---------
Co-authored-by: Jonathan Crowther <jonathan.crowther@suse.com>
Co-authored-by: Ricardo Weir <ricardo.weir@suse.com>
Co-authored-by: Michael Bolot <michael.bolot@suse.com>
pmatseykanets pushed a commit to pmatseykanets/rancher that referenced this issue Feb 8, 2024
…#372)

* Add a check for specific fields we don't want in the headers
pmatseykanets pushed a commit to pmatseykanets/rancher that referenced this issue Feb 8, 2024
…#372)

* Add a check for specific fields we don't want in the headers
pmatseykanets added a commit that referenced this issue Feb 9, 2024
* Add a check for specific fields we don't want in the headers (#372)
* Split reconcileProjectAccessToGlobalResources
* Add tests
* Check APIGroup
* Add tests and change cr client used
* [2.8] Bump API-UI version #432
* Update norman, apiserver
* Update RKE to 1.5.3
* Regenerate files after updating RKE to 1.5.3
* Update runc to 1.1.12

---------
Co-authored-by: Jonathan Crowther <jonathan.crowther@suse.com>
Co-authored-by: Ricardo Weir <ricardo.weir@suse.com>
Co-authored-by: Michael Bolot <michael.bolot@suse.com>
pmatseykanets added a commit that referenced this issue Feb 9, 2024
* Add a check for specific fields we don't want in the headers (#372)
* Split reconcileProjectAccessToGlobalResources
* Add tests
* Check APIGroup
* Add tests and change cr client used
* [2.8] Bump API-UI version #432
* Update norman, apiserver
* Update RKE to 1.5.3
* Regenerate files after updating RKE to 1.5.3

---------
Co-authored-by: Jonathan Crowther <jonathan.crowther@suse.com>
Co-authored-by: Ricardo Weir <ricardo.weir@suse.com>
Co-authored-by: Michael Bolot <michael.bolot@suse.com>
krunalhinguu pushed a commit to krunalhinguu/rancher that referenced this issue Feb 15, 2024
* Add a check for specific fields we don't want in the headers (rancher#372)
* Split reconcileProjectAccessToGlobalResources
* Add tests
* Check APIGroup
* Add tests and change cr client used
* [2.8] Bump API-UI version rancher#432
* Update norman, apiserver
* Update RKE to 1.5.3
* Regenerate files after updating RKE to 1.5.3
* Update runc to 1.1.12

---------
Co-authored-by: Jonathan Crowther <jonathan.crowther@suse.com>
Co-authored-by: Ricardo Weir <ricardo.weir@suse.com>
Co-authored-by: Michael Bolot <michael.bolot@suse.com>
krunalhinguu pushed a commit to krunalhinguu/rancher that referenced this issue Feb 15, 2024
* Add a check for specific fields we don't want in the headers (rancher#372)
* Split reconcileProjectAccessToGlobalResources
* Add tests
* Check APIGroup
* Add tests and change cr client used
* [2.8] Bump API-UI version rancher#432
* Update norman, apiserver
* Update RKE to 1.5.3
* Regenerate files after updating RKE to 1.5.3
* Update runc to 1.1.12

---------
Co-authored-by: Jonathan Crowther <jonathan.crowther@suse.com>
Co-authored-by: Ricardo Weir <ricardo.weir@suse.com>
Co-authored-by: Michael Bolot <michael.bolot@suse.com>
krunalhinguu pushed a commit to krunalhinguu/rancher that referenced this issue Feb 15, 2024
* Add a check for specific fields we don't want in the headers (rancher#372)
* Split reconcileProjectAccessToGlobalResources
* Add tests
* Check APIGroup
* Add tests and change cr client used
* [2.8] Bump API-UI version rancher#432
* Update norman, apiserver
* Update RKE to 1.5.3
* Regenerate files after updating RKE to 1.5.3
* Update runc to 1.1.12

---------
Co-authored-by: Jonathan Crowther <jonathan.crowther@suse.com>
Co-authored-by: Ricardo Weir <ricardo.weir@suse.com>
Co-authored-by: Michael Bolot <michael.bolot@suse.com>
anupama2501 pushed a commit to anupama2501/rancher that referenced this issue Mar 5, 2024
* Add a check for specific fields we don't want in the headers (rancher#372)
* Split reconcileProjectAccessToGlobalResources
* Add tests
* Check APIGroup
* Add tests and change cr client used
* [2.8] Bump API-UI version rancher#432
* Update norman, apiserver
* Update RKE to 1.5.3
* Regenerate files after updating RKE to 1.5.3
* Update runc to 1.1.12

---------
Co-authored-by: Jonathan Crowther <jonathan.crowther@suse.com>
Co-authored-by: Ricardo Weir <ricardo.weir@suse.com>
Co-authored-by: Michael Bolot <michael.bolot@suse.com>
nickwsuse pushed a commit to nickwsuse/rancher that referenced this issue Mar 14, 2024
* Add a check for specific fields we don't want in the headers (rancher#372)
* Split reconcileProjectAccessToGlobalResources
* Add tests
* Check APIGroup
* Add tests and change cr client used
* [2.8] Bump API-UI version rancher#432
* Update norman, apiserver
* Update RKE to 1.5.3
* Regenerate files after updating RKE to 1.5.3
* Update runc to 1.1.12

---------
Co-authored-by: Jonathan Crowther <jonathan.crowther@suse.com>
Co-authored-by: Ricardo Weir <ricardo.weir@suse.com>
Co-authored-by: Michael Bolot <michael.bolot@suse.com>
nickwsuse pushed a commit to nickwsuse/rancher that referenced this issue Mar 14, 2024
* Add a check for specific fields we don't want in the headers (rancher#372)
* Split reconcileProjectAccessToGlobalResources
* Add tests
* Check APIGroup
* Add tests and change cr client used
* [2.8] Bump API-UI version rancher#432
* Update norman, apiserver
* Update RKE to 1.5.3
* Regenerate files after updating RKE to 1.5.3
* Update runc to 1.1.12

---------
Co-authored-by: Jonathan Crowther <jonathan.crowther@suse.com>
Co-authored-by: Ricardo Weir <ricardo.weir@suse.com>
Co-authored-by: Michael Bolot <michael.bolot@suse.com>
nickwsuse pushed a commit to nickwsuse/rancher that referenced this issue Mar 14, 2024
* Add a check for specific fields we don't want in the headers (rancher#372)
* Split reconcileProjectAccessToGlobalResources
* Add tests
* Check APIGroup
* Add tests and change cr client used
* [2.8] Bump API-UI version rancher#432
* Update norman, apiserver
* Update RKE to 1.5.3
* Regenerate files after updating RKE to 1.5.3
* Update runc to 1.1.12

---------
Co-authored-by: Jonathan Crowther <jonathan.crowther@suse.com>
Co-authored-by: Ricardo Weir <ricardo.weir@suse.com>
Co-authored-by: Michael Bolot <michael.bolot@suse.com>
nickwsuse pushed a commit to nickwsuse/rancher that referenced this issue Apr 8, 2024
* Add a check for specific fields we don't want in the headers (rancher#372)
* Split reconcileProjectAccessToGlobalResources
* Add tests
* Check APIGroup
* Add tests and change cr client used
* [2.8] Bump API-UI version rancher#432
* Update norman, apiserver
* Update RKE to 1.5.3
* Regenerate files after updating RKE to 1.5.3

---------
Co-authored-by: Jonathan Crowther <jonathan.crowther@suse.com>
Co-authored-by: Ricardo Weir <ricardo.weir@suse.com>
Co-authored-by: Michael Bolot <michael.bolot@suse.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Issues that are defects reported by users or that we know have reached a real release
Projects
None yet
Development

No branches or pull requests

3 participants