Skip to content

Bug in v1.47.0 after allowing default bucket to be specified by the user #1174

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

Closed
elvinx opened this issue Dec 16, 2019 · 4 comments
Closed
Assignees

Comments

@elvinx
Copy link

elvinx commented Dec 16, 2019

@andremoeller thank you for the quick fix!
I am afraid this feature (#1168) introduced a bug :/ And now by default the SDK tries to create the bucket all the time

  • I upgraded to SageMaker 1.47
  • Run below lines of code
import sagemaker
role = sagemaker.get_execution_role()
  • I get the below error
    ClientError: An error occurred (AccessDenied) when calling the CreateBucket operation: Access Denied

Screen Shot 2019-12-16 at 2 22 24 PM

#1165

@elvinx elvinx changed the title Bug in 1.47 after allowing default bucket to be specified by a user Bug in v1.47.0 after allowing default bucket to be specified by a user Dec 16, 2019
@elvinx elvinx changed the title Bug in v1.47.0 after allowing default bucket to be specified by a user Bug in v1.47.0 after allowing default bucket to be specified by the user Dec 16, 2019
@andremoeller
Copy link
Contributor

@elvinx

Thanks for reporting this! We're looking into this now.

@knakad
Copy link
Contributor

knakad commented Dec 16, 2019

While we push a revert out, I believe a workaround is to specify the session with the custom bucket name as part of the session you pass in to .get_execution_role().

i.e.:

my_session = Session(..., default_bucket="my_custom_bucket")
role = sagemaker.get_execution_role(sagemaker_session=my_session)

Alternatively, you can also downgrade to a previous version to unblock yourself.

In the meantime, we're working on a version of this fix that doesn't create the bucket on init.

@ajaykarpur ajaykarpur added type: bug status: pending release The fix have been merged but not yet released to PyPI labels Dec 16, 2019
@ajaykarpur
Copy link
Contributor

Hi @elvinx, we've merged in the fix for this bug. We'll have a release out soon.

@ajaykarpur
Copy link
Contributor

The bugfix has been released. Please feel free to reach out if you have any further questions.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants