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

filters/git_last_checkout: add support for AIRBRAKE_DEPLOY_USERNAME #566

Merged
merged 1 commit into from
Apr 9, 2020

Conversation

kyrylo
Copy link
Contributor

@kyrylo kyrylo commented Apr 8, 2020

In certain environments, when a deploy starts, end deploy user might not be the
one who started the deploy. For example, if you deploy via Ansible, the script
could create a fake user who would pull the web app repo and deploy it for
you. Even though you started the script, the filter will pick up the fake user
name, since .git was created by that user.

To work around that, we introduce the AIRBRAKE_DEPLOY_USERNAME env
variable. With help of it we can forward the username from Ansible to Ruby.

@kyrylo kyrylo force-pushed the airbrake-deploy-username branch 2 times, most recently from 86e31d2 to 603f9b0 Compare April 8, 2020 10:38
In certain environments, when a deploy starts, end deploy user might not be the
one who started the deploy. For example, if you deploy via Ansible, the script
could create a fake user who would pull the web app repo and deploy it for
you. Even though you started the script, the filter will pick up the fake user
name, since `.git` was created by that user.

To work around that, we introduce the `AIRBRAKE_DEPLOY_USERNAME` env
variable. With help of it we can forward the username from Ansible to Ruby.
@kyrylo kyrylo force-pushed the airbrake-deploy-username branch from 603f9b0 to 631eda7 Compare April 8, 2020 10:39
@kyrylo kyrylo merged commit 11c4670 into master Apr 9, 2020
@kyrylo kyrylo deleted the airbrake-deploy-username branch April 10, 2020 09:37
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.

1 participant