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

iam role default support added #8100

Merged
merged 1 commit into from May 6, 2016

Conversation

Projects
None yet
3 participants
@bilalyasar
Copy link
Collaborator

commented May 5, 2016

you can find explanation here #7087
fixes #7087

@pveentjer

This comment has been minimized.

Copy link
Member

commented May 5, 2016

How is this tested? Did you try it on amazon? I know that adding a unit test for it will be a difficult problem.

@bilalyasar

This comment has been minimized.

Copy link
Collaborator Author

commented May 6, 2016

@pveentjer yes, i tried in amazon manually. After merging, we can also ask the user for trying.

@pveentjer

This comment has been minimized.

Copy link
Member

commented May 6, 2016

👍

@pveentjer pveentjer merged commit e687d8a into hazelcast:maintenance-3.x May 6, 2016

1 check passed

default 10884 tests run, 39 skipped, 0 failed.
Details
@mesutcelik

This comment has been minimized.

Copy link
Contributor

commented May 27, 2016

@bilalyasar Can you please forward port?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.