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

Revert "(BKR-967) Add :disable_analytics option" #39

Merged
merged 1 commit into from
Nov 22, 2016

Conversation

james-stocks
Copy link

This reverts commit c9e2565.

This commit is not useful because it is blocking analytics from the console host;
but analytics are sent from the user browser and not the console host.

This reverts commit c9e2565.

This commit is not useful because it is blocking analytics from the console host;
but analytics are sent from the user browser and not the console host.
@puppetlabs-jenkins
Copy link
Contributor

Can one of the admins verify this patch?

@puppetlabs-jenkins
Copy link
Contributor

Refer to this link for build results (access rights to CI server needed):
http://jenkins-beaker.delivery.puppetlabs.net//job/qe_beaker-pe_btc-intn/134/

@james-stocks
Copy link
Author

Acceptance test failed only on Windows 2008 - it doesn't appear to be caused by this change

@james-stocks
Copy link
Author

@puppetlabs-jenkins retest this please

@puppetlabs-jenkins
Copy link
Contributor

Refer to this link for build results (access rights to CI server needed):
http://jenkins-beaker.delivery.puppetlabs.net//job/qe_beaker-pe_btc-intn/136/

@kevpl kevpl merged commit 2d17d83 into puppetlabs:master Nov 22, 2016
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.

3 participants