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

aws-toolbox\Get-ATEBInstanceLogs -EnvironmentName name #18

Closed
kehtabp opened this issue Jun 5, 2019 · 2 comments
Closed

aws-toolbox\Get-ATEBInstanceLogs -EnvironmentName name #18

kehtabp opened this issue Jun 5, 2019 · 2 comments
Assignees
Labels
bug Something isn't working

Comments

@kehtabp
Copy link

kehtabp commented Jun 5, 2019

image

Text version:

aws-toolbox\Get-ATEBInstanceLogs -EnvironmentName css-exbox-prod
Getting EB environment details...
Checking instances for SSM...
Sending SSM command to i-0547c72cda6ebd86c, i-0fac48d87d29b2e65
Submitted command with ID c3bf2691-cafa-4f9f-86bd-8831cd11c1fd and waiting for status...
Collecting results from S3...
https://s3.eu-west-1.amazonaws.com/aws-toolbox-workspace-eu-west-1-612155760304/ssm-run-command/c3bf2691-cafa-4f9f-86bd
-8831cd11c1fd/i-0547c72cda6ebd86c/awsrunPowerShellScript/0.awsrunPowerShellScript/stdout: Not an S3 URL
At C:\Users\aleksandrovk\Documents\WindowsPowerShell\Modules\aws-toolbox\0.14.0\Private\S3\Split-S3Url.ps1:27 char:9
+         throw "$($S3Url): Not an S3 URL"
+         ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : OperationStopped: (https://s3.eu-w...: Not an S3 URL:String) [], RuntimeException
    + FullyQualifiedErrorId : https://s3.eu-west-1.amazonaws.com/aws-toolbox-workspace-eu-west-1-612155760304/ssm-run-
   command/c3bf2691-cafa-4f9f-86bd-8831cd11c1fd/i-0547c72cda6ebd86c/awsrunPowerShellScript/0.awsrunPowerShellScript/s
  tdout: Not an S3 URL
@fireflycons fireflycons self-assigned this Jun 5, 2019
@fireflycons fireflycons added the bug Something isn't working label Jun 5, 2019
@fireflycons
Copy link
Owner

Interesting one @kehtabp

If you have a look at https://aws.amazon.com/blogs/aws/amazon-s3-path-deprecation-plan-the-rest-of-the-story/ in section Old vs New you'll see that 'regional endpoint' paths should be formed e.g. s3-eu-west-1 with a - after the s3. This was clearly the case till when you last saw it work (me, yesterday).

Seems they've subtly changed it so it's now s3.eu-west-1 and that's not referred to at all in that web page :-(

So I'll fix it to look for all forms on that page, plus s3. and hopefully that'll future proof it.

fireflycons added a commit that referenced this issue Jun 5, 2019
@kehtabp
Copy link
Author

kehtabp commented Jun 6, 2019 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants