Skip to content
This repository was archived by the owner on Mar 5, 2025. It is now read-only.

Update ci.md #1082

Merged
merged 1 commit into from
Feb 14, 2017
Merged

Update ci.md #1082

merged 1 commit into from
Feb 14, 2017

Conversation

danepowell
Copy link
Contributor

@danepowell danepowell commented Feb 14, 2017

This was last touched by @mikemadison13 in #794 / #795, which recommended using a personal account because Cloud no longer allowed plus signs. However, using a personal account is not very secure since it would allow anyone on a project to bypass access control and access any subscription associated with your account. It also means that deployments will begin to fail unexpectedly if you are removed from the project.

I just verified that you can create new Cloud accounts using emails with plus signs, as well as invite these accounts to any team, so I think we should go back to recommending the use of a dedicated account.

Note that we have internal tickets to track improvements to this process (NN-12088, CSPF-191).

@mikemadison13
Copy link
Contributor

@danepowell the cloud team recently fixed this, I believe

@danepowell
Copy link
Contributor Author

Awesome that would make sense. Thanks.

@grasmash grasmash merged commit 362da78 into 8.x Feb 14, 2017
@grasmash grasmash deleted the danepowell-patch-6 branch February 15, 2017 02:09
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants