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

BLT-794: updating documentation to work around acquia cloud email san… #795

Merged
merged 1 commit into from
Dec 15, 2016

Conversation

mikemadison13
Copy link
Contributor

Changes proposed:

  • Updated documentation to reflect work around given no ability to create extra cloud accounts with + in the email address.
  • Added clarification regarding TA account access and Build failures
  • Added clarification for drush sql-sync and rsync requirements in known host configuration

1. Add the same public SSH key to the "Deployment Keys" section on your project's GitHub settings page, located at `https://github.com/acquia-pso/[project-name]/settings/keys`. **Note: You may not have direct access to these settings if you do not have administrative control over your repository.**
1. Add the _private SSH key_ to your project's Travis CI settings located at `https://magnum.travis-ci.com/acquia-pso/[project-name]/settings`.
1. Add your cloud git repository to the remotes section of your project.yml file:

remotes:
- example@svn-14671.prod.hosting.acquia.com:example.git`
Note: if planning on executing any drush sql-syncs/rsyncs between the cloud and your environment, also add the test/stage server host here and ensure that your account has the senior developer or team lead role (or appropriate custom access in the Acquia Cloud)..
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please remove trailing period


1. Commits or merges to the develop branch on GitHub should now trigger a fully built artifact to be deployed to your specified remotes.
1. **If the TA account is removed from the project, the Travis Builds for that project will begin to fail.**
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's not use acquia titles (TA) in docs.

@grasmash grasmash merged commit 70847b9 into acquia:8.x Dec 15, 2016
@mikemadison13 mikemadison13 deleted the BLT-794 branch December 16, 2016 21:29
@danepowell danepowell mentioned this pull request Feb 14, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants