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.

Sign up for GitHub

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

Jump to bottom

[STABILITY] Permission denied issue on builder-api-proxy directory #3521

Closed
chefsalim opened this issue Oct 5, 2017 · 4 comments · Fixed by #4193 or #4366
Closed

[STABILITY] Permission denied issue on builder-api-proxy directory #3521

chefsalim opened this issue Oct 5, 2017 · 4 comments · Fixed by #4193 or #4366
Assignees
@christophermaier
Labels
Focus:Supervisor Related to the Habitat Supervisor (core/hab-sup) component Type: Bug Issues that describe broken functionality Type:Stability

Comments

@chefsalim
Copy link
Contributor

chefsalim commented Oct 5, 2017 •
edited
Loading

We need to fix the issue where the supervisor sets the service directories to be owned by root, when they should be owned by the 'hab' user (nginx scenario).
@chefsalim chefsalim added this to the Pre-Neighborhoods milestone Oct 5, 2017
@chefsalim chefsalim self-assigned this Oct 6, 2017
@chefsalim
Copy link
Contributor Author

Re-opening as the hook fix does not seem to work.
Also we should handle the permission change for the builder api data folder.

@elliott-davis elliott-davis removed their assignment Oct 25, 2017
@chefsalim chefsalim self-assigned this Oct 27, 2017
@chefsalim chefsalim assigned baumanj and unassigned chefsalim and baumanj Nov 3, 2017
@elliott-davis elliott-davis self-assigned this Nov 17, 2017
@elliott-davis elliott-davis mentioned this issue Nov 17, 2017
Merged
@chefsalim
Copy link
Contributor Author

Re-opening as this is still occuring

@chefsalim chefsalim reopened this Nov 28, 2017
@chefsalim chefsalim removed this from the Pre-Neighborhoods milestone Nov 28, 2017
@chefsalim chefsalim changed the title Permission denied issue on builder-api-proxy directory [STABILITY] Permission denied issue on builder-api-proxy directory Nov 28, 2017
@christophermaier christophermaier self-assigned this Nov 30, 2017
christophermaier added a commit that referenced this issue Dec 1, 2017
@christophermaier
Ensure permissions are set on the config directory
01fe7a1 
NOTE: This is a temporary bandaid for the micro-outages we've been
having with the API server every time the service restarts. The real
fix lies in how the Supervisor starts services (and more specifically,
which user they are started by). Addressing that should (partially)
unlock new behaviors for the Supervisor (like the ability to more
sanely run as a non-root user).

Fixes: #3521

Signed-off-by: Christopher Maier <cmaier@chef.io>
@christophermaier christophermaier mentioned this issue Dec 1, 2017
Merged
@tashimi tashimi reopened this Dec 19, 2017
@tashimi
Copy link
Contributor

tashimi commented Dec 19, 2017

pretty sure this issue is still happening, and known to be so; Chris had to fix the permissions in acceptance in builder-api-proxy manually today

@tashimi tashimi mentioned this issue Dec 19, 2017
Closed
@christophermaier christophermaier mentioned this issue Dec 20, 2017
Merged
@burtlo
Copy link

burtlo commented Jan 5, 2018 •
edited
Loading

Thank you very much for fixing this!

giphy-tumblr

burtlo pushed a commit to learn-chef/hab-two-tier that referenced this issue Jan 5, 2018
Removes the reconfigure hook as this is now addressed
be2a337 
SEE Issue: habitat-sh/habitat#3521

Signed-off-by: Franklin Webber <franklin@chef.io>
@christophermaier christophermaier added Focus:Supervisor Related to the Habitat Supervisor (core/hab-sup) component Type: Bug Issues that describe broken functionality and removed A-supervisor labels Jul 24, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@christophermaier christophermaier

Labels
Focus:Supervisor Related to the Habitat Supervisor (core/hab-sup) component Type: Bug Issues that describe broken functionality Type:Stability
Projects
None yet
Milestone
No milestone
7 participants
@christophermaier @burtlo @tashimi @elliott-davis @baumanj @eeyun @chefsalim