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

Slack Notify with Drush Site Audit on Deploy #77

Open
wants to merge 37 commits into
base: master
Choose a base branch
from

Conversation

jordan8037310
Copy link
Contributor

Drush Site Audit on deploy.

image

  • @todo: Duplicate to HipChat
  • This is implemented as a separate script, didn't want to assume everyone would ALWAYS want to run the site audit. Also this report is quite verbose, so I think people should opt-in if they want to pollute their Slack channel.
  • This is grouped under /slack_notification example, but could be arguably moved to it's own example as drush_site_audit or something similar. Concern there is duplicating Slack notification markup further. Building this makes me think a slack_helper.php would be easier to include basic functionality across multiple scripts/quicksilver workflow tasks. It looks like this was recently considered in combine-slack branch and deemed unnecessary when slack_lib.inc was merged into the main file. Perhaps this is cause to reconsider.

PS - Thanks to conversation with @JessiFischer this week at DrupalCon—this seemed worthwhile to implement & make sure no mistakes were made post-deploy for some of our sites.

jordan8037310 and others added 30 commits May 12, 2016 22:12
* @todo: Duplicate to HipChat
* This is implemented as a separate script, didn't want to assume everyone would ALWAYS want to run the site audit. Also this report is quite verbose, so I think people should opt-in if they want to pollute their Slack channel.
* This is grouped under /slack_notification example, but could be arguably moved to it's own example as drush_site_audit or something similar. Concern there is duplicating Slack notification markup further. Makes me think a slack_helper.php would be easier to include basic functionality across multiple scripts/quicksilver workflow tasks.
Also pretty things up a little bit.
Site and Multidev Creation
…w-hooks

Add new hooks to Slack example README
- Nested DOCROOT (web_root)
- protected_web_paths
- php_version

New Quicksilver Hooks:
* Site creation (deploy_product)
* Multidev creation (create_cloud_development_environment)

Hold for release
…-site-configuration-new-quicksilver-hooks

Add new features to example.pantheon.yml
typo fix

Updating yml example

title fix
…eon-systems#87)

* Add Solr Power indexing on multidev creation example

* Repost schema before indexing

* Clarify WordPress for this example

* Remove duplicate directory
…elly-patch-1

Update slack_notification.php
…rsion

Add drush_version to example.pantheon.yml
…pdex

* Set Apdex T values on multidev creation
* Closes pantheon-systems#86
Fix spelling of "snippet" in new_relic_deploy
allenfear and others added 7 commits October 23, 2016 09:56
Specifying a channel can mean the default one from the webhook isn't used. Also link to some spiffy new docs they have.
* @todo: Duplicate to HipChat
* This is implemented as a separate script, didn't want to assume everyone would ALWAYS want to run the site audit. Also this report is quite verbose, so I think people should opt-in if they want to pollute their Slack channel.
* This is grouped under /slack_notification example, but could be arguably moved to it's own example as drush_site_audit or something similar. Concern there is duplicating Slack notification markup further. Makes me think a slack_helper.php would be easier to include basic functionality across multiple scripts/quicksilver workflow tasks.
…silver-examples into site-audit-on-deploy

* 'site-audit-on-deploy' of github.com:jordan8037310/quicksilver-examples:
  Drush Site Audit on deploy.
  Slack Attachments Title should really be set per-case
  'Title' uses better formatting than pretext in Slack Attachments. Emojis display as well.
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.

10 participants