Jira (PUP-7102) Insufficient logging when encountering 'permission denied' on environmentpath

8 views
Skip to first unread message

Jeremy Barlow (JIRA)

unread,
Jan 13, 2017, 5:14:03 PM1/13/17
to puppe...@googlegroups.com
Jeremy Barlow updated an issue
 
Puppet / Bug PUP-7102
Insufficient logging when encountering 'permission denied' on environmentpath
Change By: Jeremy Barlow
Affects Version/s: PUP 4.8.1
Add Comment Add Comment
 
This message was sent by Atlassian JIRA (v6.4.14#64029-sha1:ae256fe)
Atlassian logo

Jeremy Barlow (JIRA)

unread,
Jan 13, 2017, 5:14:03 PM1/13/17
to puppe...@googlegroups.com
Jeremy Barlow moved an issue
Change By: Jeremy Barlow
Affects Version/s: SERVER 2.7.2
Component/s: Puppet Server
Key: SERVER PUP - 1716 7102
Project: Puppet  Server

Jeremy Barlow (JIRA)

unread,
Jan 13, 2017, 5:15:02 PM1/13/17
to puppe...@googlegroups.com

Henrik Lindberg (JIRA)

unread,
Jan 18, 2017, 7:02:02 AM1/18/17
to puppe...@googlegroups.com
Henrik Lindberg updated an issue
 
Change By: Henrik Lindberg
Fix Version/s: PUP 5.0.0

Henrik Lindberg (JIRA)

unread,
Jan 18, 2017, 7:03:02 AM1/18/17
to puppe...@googlegroups.com
Henrik Lindberg updated an issue
Change By: Henrik Lindberg
Story Points: 2
Team: Puppet Developer Experience
Sprint: Puppet 5.0

Henrik Lindberg (JIRA)

unread,
Jan 18, 2017, 7:04:02 AM1/18/17
to puppe...@googlegroups.com
Henrik Lindberg updated an issue
Change By: Henrik Lindberg
Sprint: Puppet 5.0 PDE 2017-02-22

Henrik Lindberg (JIRA)

unread,
Jan 18, 2017, 7:04:02 AM1/18/17
to puppe...@googlegroups.com
Henrik Lindberg updated an issue
Change By: Henrik Lindberg
Sprint: PDE 2017-02- 22 08

Henrik Lindberg (JIRA)

unread,
Jan 26, 2017, 5:52:02 AM1/26/17
to puppe...@googlegroups.com
Henrik Lindberg assigned an issue to Thomas Hallgren
Change By: Henrik Lindberg
Assignee: Thomas Hallgren

Thomas Hallgren (JIRA)

unread,
Jan 27, 2017, 6:37:24 AM1/27/17
to puppe...@googlegroups.com
Thomas Hallgren commented on Bug PUP-7102
 
Re: Insufficient logging when encountering 'permission denied' on environmentpath

Not sure where the fix should go at this time. I've asked Moses Mendoza, who placed a copy of the gem in the puppet repo.

Thomas Hallgren (JIRA)

unread,
Jan 27, 2017, 8:08:03 AM1/27/17
to puppe...@googlegroups.com
Thomas Hallgren updated an issue
 
Change By: Thomas Hallgren
Comment: Not sure where the fix should go at this time. I've asked [~moses], who placed a copy of the gem in the puppet repo.

Thomas Hallgren (JIRA)

unread,
Jan 27, 2017, 5:23:15 PM1/27/17
to puppe...@googlegroups.com
Thomas Hallgren assigned an issue to Unassigned
Change By: Thomas Hallgren
Assignee: Thomas Hallgren

Henrik Lindberg (JIRA)

unread,
Jan 31, 2017, 9:23:03 AM1/31/17
to puppe...@googlegroups.com
Henrik Lindberg assigned an issue to Thomas Hallgren
Change By: Henrik Lindberg
Assignee: Thomas Hallgren

Eric Thompson (JIRA)

unread,
Jan 31, 2017, 1:57:03 PM1/31/17
to puppe...@googlegroups.com
Eric Thompson updated an issue
Change By: Eric Thompson
QA Risk Assessment: Needs Assessment Automate

Eric Thompson (JIRA)

unread,
Jan 31, 2017, 1:57:03 PM1/31/17
to puppe...@googlegroups.com
Eric Thompson updated an issue
Change By: Eric Thompson
QA Risk Assessment Reason: not covered by unit tests; automate, medium priority

Henrik Lindberg (JIRA)

unread,
Jan 31, 2017, 8:08:03 PM1/31/17
to puppe...@googlegroups.com
Henrik Lindberg updated an issue
Change By: Henrik Lindberg
Acceptance Criteria: When  multiple paths are specified for  resolution of an environment name to path takes place and the effective user does not have access to  a  configuration option like '  path included in the environmentpath ',  puppet should  raise an error when the access is denied because of permissions. The rationale for this is that  it  is impossible to tell if the environment is missing or not in the expected location. Therefore a decision that a) it did not exist, or b) a directory found later in the environmentpath  should be  easy  used are wrong because it is inconclusive.

Therefore:

If puppet can sudo  to  determine which path was picked during  check the actual existence without  an  agent run  auth error it should do so to allow the search to continue past a bona fide "not found" ,  and why. 

Specifically, in this case,  that with lowered access rights would look like  a permission denied  message should  error. (i.e.  make  it into  an inconclusive error conclusive if possible).

Only a bona fide "file not found" should mean that  the  puppetserver logs  search continues .  Only if the search ends error free and with "production" not found should an attempt be made to create that environment.

Henrik Lindberg (JIRA)

unread,
Jan 31, 2017, 8:11:02 PM1/31/17
to puppe...@googlegroups.com
Henrik Lindberg commented on Bug PUP-7102
 
Re: Insufficient logging when encountering 'permission denied' on environmentpath

Thomas Hallgren Acceptance criteria updated.

The rationale is that the big problem here is errors (other than "not found") are swallowed. It should simply stop and report that. Warnings etc. still lead to bad behavior, and you would need to know that you must look in the logs. Warnings are not propagated to the user in most scenarios (they are in the apply case since you are seeing the log output, but not in an agent -t run against the master).

It is especially troublesome if there is a "production" env with permission error as the system now takes that as "not found" and it goes of to create the default production environment. That in turn can lead to horrible effects with nodes getting empty catalogs, or strange catalogs. Hence - it should simply error out.

Craig Gomes (JIRA)

unread,
Feb 8, 2017, 4:20:05 PM2/8/17
to puppe...@googlegroups.com
Craig Gomes updated an issue
 
Change By: Craig Gomes
Sprint: PDE 2017-02-08 , PDE 2017-02-22

Thomas Hallgren (JIRA)

unread,
Feb 20, 2017, 6:57:03 AM2/20/17
to puppe...@googlegroups.com
Thomas Hallgren assigned an issue to Unassigned
Change By: Thomas Hallgren
Assignee: Thomas Hallgren

Henrik Lindberg (JIRA)

unread,
Feb 21, 2017, 4:54:02 AM2/21/17
to puppe...@googlegroups.com
Henrik Lindberg assigned an issue to Thomas Hallgren
Change By: Henrik Lindberg
Assignee: Thomas Hallgren

Thomas Hallgren (JIRA)

unread,
Feb 21, 2017, 5:07:02 AM2/21/17
to puppe...@googlegroups.com
Thomas Hallgren assigned an issue to Unassigned
Change By: Thomas Hallgren
Assignee: Thomas Hallgren

Thomas Hallgren (JIRA)

unread,
Feb 21, 2017, 5:28:03 AM2/21/17
to puppe...@googlegroups.com

Craig Gomes (JIRA)

unread,
Feb 22, 2017, 4:12:06 PM2/22/17
to puppe...@googlegroups.com
Craig Gomes updated an issue
Change By: Craig Gomes
Sprint: PDE 2017-02-08, PDE 2017-02-22 , PDE 2017-03-08

Josh Cooper (JIRA)

unread,
Feb 23, 2017, 1:04:02 AM2/23/17
to puppe...@googlegroups.com
Josh Cooper commented on Bug PUP-7102
 
Re: Insufficient logging when encountering 'permission denied' on environmentpath

One thing I noticed is that if the permissions are incorrect on one of the ancestor directories then puppetserver will fail to start:

06:01:52     Feb 21 06:01:21 rbjkogu91d1gagw systemd[1]: Starting puppetserver Service...
 
06:01:52     Feb 21 06:01:21 rbjkogu91d1gagw puppetserver[13389]: OpenJDK 64-Bit Server VM warning: ignoring option MaxPermSize=256m; support was removed in 8.0
 
06:01:52     Feb 21 06:01:50 rbjkogu91d1gagw puppetserver[13389]: Errno::EACCES: Permission denied - /tmp/lookup_test_dir.hZDf2k
 
06:01:52     Feb 21 06:01:50 rbjkogu91d1gagw puppetserver[13389]: valid_directory? at /opt/puppetlabs/puppet/lib/ruby/vendor_ruby/puppet/environments.rb:242

This is different than the compilation will fail with HTTP 500 during catalog compilation. I'm not sure if that was because the test created a temp codedir directory owned by root:root and mode 700 (as opposed to a directory at a lower level, like $codedir/environments/<environment>). If the behavior is expected (for puppetserver to fail to start), then we should make sure to release note it.

Henrik Lindberg (JIRA)

unread,
Feb 23, 2017, 8:25:02 AM2/23/17
to puppe...@googlegroups.com

Josh Cooper that does not sound like a nice way to inform the user about the problem. Ping Thomas Hallgren

Henrik Lindberg (JIRA)

unread,
Feb 27, 2017, 11:05:03 AM2/27/17
to puppe...@googlegroups.com
Henrik Lindberg updated an issue
 
Change By: Henrik Lindberg
Release Notes Summary: When access was denied to an environment directory puppet skipped that in favor of the next (if any) matching environment on the environment path. This led to hard to detect problems. Now, an error is raised indicating if an environment is inaccessible due to a authorization issue.
Release Notes: Bug Fix

John Duarte (JIRA)

unread,
Feb 28, 2017, 1:09:02 AM2/28/17
to puppe...@googlegroups.com
John Duarte commented on Bug PUP-7102
 
Re: Insufficient logging when encountering 'permission denied' on environmentpath

I can confirm Josh Cooper's observation. The Eeeno::EACCES puppetserver error is currently manifesting itself in recent puppet#master CI runs causing all cells to to red. Reverting commit c639c94b causes the puppetserver error to go away.

Thomas Hallgren (JIRA)

unread,
Feb 28, 2017, 4:05:16 AM2/28/17
to puppe...@googlegroups.com

I'm fine with reverting this. We need to investigate all places where the change might have a negative impact. I still think it's a good idea to not proceed unless the access permissions are correct, but the user must be notified in a way that makes sense, no matter what the use-case is.

Henrik Lindberg (JIRA)

unread,
Feb 28, 2017, 5:52:02 AM2/28/17
to puppe...@googlegroups.com

+1 to reverting - it reverts back to how things have been for a very long time

John Duarte (JIRA)

unread,
Feb 28, 2017, 10:25:12 AM2/28/17
to puppe...@googlegroups.com
John Duarte commented on Bug PUP-7102

PR created to revert.

Josh Cooper (JIRA)

unread,
Feb 28, 2017, 11:38:07 AM2/28/17
to puppe...@googlegroups.com
Josh Cooper commented on Bug PUP-7102

I think the revert is fine for now, but I agree with Thomas Hallgren that it isn't good to swallow errors and continue along.

I think the failure to start puppetserver is mostly an issue in CI because we frequently create a temp codedir and point the server at that. In practice though, puppetserver uses a codedir that was created by packaging and has the correct permissions. The part that users get wrong is creating a per-environment directory, but leaving it owned by root, and not readable by the puppet user, or forgetting to create the directory. I think failing to start, with an improved error message, is the correct behavior.

Side node, I think many acceptance tests were copy/pasted to create a tempdir with the wrong permissions, and later after puppetserver was running, set the permissions correctly. For example, we fixed this for one test in 057def0c3. So part of the investigation should be to fix the acceptance tests to create the tempdir with the correct permissions (puppet:puppet:755) before calling with_puppet_running_on. I also found that the above acceptance tests started puppetserver, but only ran puppet apply commands, so puppetserver was unnecessary. So keep an eye out for tests that can be streamlined.

Craig Gomes (JIRA)

unread,
Mar 16, 2017, 10:33:06 AM3/16/17
to puppe...@googlegroups.com
Craig Gomes updated an issue
 
Change By: Craig Gomes
Sprint: PDE 2017-02-08, PDE 2017-02-22, PDE 2017-03-08 , PDE 2017-03-22

Henrik Lindberg (JIRA)

unread,
Mar 22, 2017, 7:14:07 PM3/22/17
to puppe...@googlegroups.com
Henrik Lindberg updated an issue
Change By: Henrik Lindberg
Sprint: PDE 2017-02-08, PDE 2017-02-22, PDE 2017-03-08, PDE 2017-03-22 , PDE 2017-04-05

Thomas Hallgren (JIRA)

unread,
Apr 5, 2017, 7:35:02 AM4/5/17
to puppe...@googlegroups.com
Thomas Hallgren commented on Bug PUP-7102
 
Re: Insufficient logging when encountering 'permission denied' on environmentpath

Josh Cooper, is the CI issue that prevented this ticket from going in being addressed? If so, is there a ticket for it?

Josh Cooper (JIRA)

unread,
Apr 5, 2017, 11:38:02 AM4/5/17
to puppe...@googlegroups.com
Josh Cooper commented on Bug PUP-7102

Thomas Hallgren We still need to audit/ensure existing acceptance tests are not starting puppetserver with a conf/var/etc dir with incorrect permissions otherwise we'll see the same errors when the change is re-reverted. I was thinking that effort would be part of this ticket (since this ticket is about ensuring we don't swallow errors when that situation occurs). You'll want to look for tests that rely on with_puppet_running_on and point the puppetserver to a temp directory which doesn't have owner/group set to puppet:puppet.

Henrik Lindberg (JIRA)

unread,
Apr 5, 2017, 4:28:05 PM4/5/17
to puppe...@googlegroups.com
Henrik Lindberg updated an issue
Change By: Henrik Lindberg
Sprint: PDE 2017-02-08, PDE 2017-02-22, PDE 2017-03-08, PDE 2017-03-22, PDE 2017-04-05 , PDE 2017-04-19

Eric Sorenson (JIRA)

unread,
Apr 20, 2017, 9:07:02 PM4/20/17
to puppe...@googlegroups.com
Eric Sorenson commented on Bug PUP-7102
 
Re: Insufficient logging when encountering 'permission denied' on environmentpath

It'd be nice to have this but it's not a blocker for 5.0.0.

Eric Sorenson (JIRA)

unread,
Apr 20, 2017, 9:08:02 PM4/20/17
to puppe...@googlegroups.com
Eric Sorenson updated an issue
 
Change By: Eric Sorenson
Fix Version/s: PUP 5.0.0
Fix Version/s: PUP 5.y

Henrik Lindberg (JIRA)

unread,
Apr 25, 2017, 5:14:04 AM4/25/17
to puppe...@googlegroups.com
Henrik Lindberg updated an issue
Change By: Henrik Lindberg
Sprint: PDE 2017-02-08, PDE 2017-02-22, PDE 2017-03-08, PDE 2017-03-22, PDE 2017-04-05, PDE 2017-04-19 , PDE 2017-05-03

Henrik Lindberg (JIRA)

unread,
Apr 25, 2017, 5:17:03 AM4/25/17
to puppe...@googlegroups.com
Henrik Lindberg updated an issue
Change By: Henrik Lindberg
Sprint: PDE 2017-02-08, PDE 2017-02-22, PDE 2017-03-08, PDE 2017-03-22, PDE 2017-04-05, PDE 2017-04-19, PDE 2017-05- 03 17

Henrik Lindberg (JIRA)

unread,
May 2, 2017, 10:28:06 AM5/2/17
to puppe...@googlegroups.com
Henrik Lindberg updated an issue
Change By: Henrik Lindberg
Sprint: PDE 2017-02-08, PDE 2017-02-22, PDE 2017-03-08, PDE 2017-03-22, PDE 2017-04-05, PDE 2017-04-19, PDE 2017-05- 17 31

Branan Riley (JIRA)

unread,
May 16, 2017, 5:26:04 PM5/16/17
to puppe...@googlegroups.com
Branan Riley updated an issue
Change By: Branan Riley
Labels: triaged

Branan Riley (JIRA)

unread,
May 16, 2017, 5:26:05 PM5/16/17
to puppe...@googlegroups.com
Branan Riley updated an issue
Change By: Branan Riley
Team: Puppet Developer Experience Agent

Henrik Lindberg (JIRA)

unread,
May 31, 2017, 8:12:05 AM5/31/17
to puppe...@googlegroups.com
Henrik Lindberg updated an issue
Change By: Henrik Lindberg
Sprint: PDE 2017-02-08, PDE 2017-02-22, PDE 2017-03-08, PDE 2017-03-22, PDE 2017-04-05, PDE 2017-04-19 , PDE 2017-06-07

Henrik Lindberg (JIRA)

unread,
May 31, 2017, 8:12:07 AM5/31/17
to puppe...@googlegroups.com
Henrik Lindberg updated an issue
Change By: Henrik Lindberg
Sprint: PDE  Triage, PDE  2017-02-08, PDE 2017-02-22, PDE 2017-03-08, PDE 2017-03-22, PDE 2017-04-05, PDE 2017-04-19

Henrik Lindberg (JIRA)

unread,
May 31, 2017, 9:01:13 AM5/31/17
to puppe...@googlegroups.com

Henrik Lindberg (JIRA)

unread,
May 31, 2017, 9:01:18 AM5/31/17
to puppe...@googlegroups.com
Henrik Lindberg updated an issue
Change By: Henrik Lindberg
Sprint: PDE 2017-02-08, PDE 2017-02-22, PDE 2017-03-08, PDE 2017-03-22, PDE 2017-04-05,  Forge - To Accept,  PDE 2017-04-19

Geoff Nichols (JIRA)

unread,
Jun 20, 2017, 4:16:03 PM6/20/17
to puppe...@googlegroups.com
Geoff Nichols updated an issue
Change By: Geoff Nichols
Sprint: PDE 2017-02-08, PDE 2017-02-22, PDE 2017-03-08, PDE 2017-03-22, PDE 2017-04-05,  Forge - To Accept,  PDE 2017-04-19

Thomas Hallgren (JIRA)

unread,
Jun 27, 2017, 6:02:03 AM6/27/17
to puppe...@googlegroups.com
Thomas Hallgren assigned an issue to Thomas Hallgren
Change By: Thomas Hallgren
Assignee: Thomas Hallgren
Sub-team: Language
Sprint: PDE 2017-02-08, PDE 2017-02-22, PDE 2017-03-08, PDE 2017-03-22, PDE 2017-04-05, PDE 2017-04-19 , Agent 2017-06-28

Thomas Hallgren (JIRA)

unread,
Jun 27, 2017, 6:13:03 AM6/27/17
to puppe...@googlegroups.com
Thomas Hallgren updated an issue
Change By: Thomas Hallgren
Fix Version/s: PUP 5.y
Fix Version/s: PUP 5.1.0

Thomas Hallgren (JIRA)

unread,
Jun 27, 2017, 6:57:03 AM6/27/17
to puppe...@googlegroups.com
Thomas Hallgren commented on Bug PUP-7102
 
Re: Insufficient logging when encountering 'permission denied' on environmentpath

Josh Cooper, in the last PR, I included an acceptance test using with_puppet_running_on that verifies that the agent gets a proper error back when the master is unable to read the environment due to permission problems. It does however not include any specifics for puppetserver, simply because I have no clue how to write or run such tests. It's my hope that my test can be amended if need be, or that an additional test can be added in a separate PR authored by someone from CI.

Thomas Hallgren (JIRA)

unread,
Jun 27, 2017, 6:59:02 AM6/27/17
to puppe...@googlegroups.com
Thomas Hallgren assigned an issue to Josh Cooper
 
Change By: Thomas Hallgren
Assignee: Thomas Hallgren Josh Cooper

Geoff Nichols (JIRA)

unread,
Jun 28, 2017, 4:20:05 PM6/28/17
to puppe...@googlegroups.com
Geoff Nichols updated an issue
Change By: Geoff Nichols
Sprint: PDE 2017-02-08, PDE 2017-02-22, PDE 2017-03-08, PDE 2017-03-22, PDE 2017-04-05, PDE 2017-04-19, Agent 2017-06-28 , Agent 2017-07-12

Geoff Nichols (JIRA)

unread,
Jul 12, 2017, 1:26:04 PM7/12/17
to puppe...@googlegroups.com
Geoff Nichols updated an issue
Change By: Geoff Nichols
Sprint: PDE 2017-02-08, PDE 2017-02-22, PDE 2017-03-08, PDE 2017-03-22, PDE 2017-04-05, PDE 2017-04-19, Agent 2017-06-28, Agent 2017-07-12 , Agent Hold

Geoff Nichols (JIRA)

unread,
Jul 19, 2017, 3:21:03 AM7/19/17
to puppe...@googlegroups.com
Geoff Nichols updated an issue
Change By: Geoff Nichols
Sprint: PDE 2017-02-08, PDE 2017-02-22, PDE 2017-03-08, PDE 2017-03-22, PDE 2017-04-05, PDE 2017-04-19, Agent  N+1, Agent  2017-06-28, Agent 2017-07-12 , Agent Hold

Josh Cooper (JIRA)

unread,
Jul 31, 2017, 7:25:02 PM7/31/17
to puppe...@googlegroups.com
Josh Cooper assigned an issue to Unassigned
Change By: Josh Cooper
Assignee: Josh Cooper

Karen Van der Veer (JIRA)

unread,
Aug 4, 2017, 2:39:09 PM8/4/17
to puppe...@googlegroups.com
Karen Van der Veer updated an issue
Change By: Karen Van der Veer
Sprint: PDE 2017-02-08, PDE 2017-02-22, PDE 2017-03-08, PDE 2017-03-22, PDE 2017-04-05, PDE 2017-04-19, Platform Core 2017-09- 05 19 , Agent 2017-06-28, Agent 2017-07-12

Karen Van der Veer (JIRA)

unread,
Aug 4, 2017, 2:40:04 PM8/4/17
to puppe...@googlegroups.com
Karen Van der Veer updated an issue
Change By: Karen Van der Veer
Sprint: PDE 2017-02-08, PDE 2017-02-22, PDE 2017-03-08, PDE 2017-03-22, PDE 2017-04-05, PDE 2017-04-19,  Platform Core 2017-09-19,  Agent 2017-06-28, Agent 2017-07-12 , Platform Core Grooming

Karen Van der Veer (JIRA)

unread,
Aug 7, 2017, 6:59:03 PM8/7/17
to puppe...@googlegroups.com
Karen Van der Veer updated an issue
Change By: Karen Van der Veer
Sprint: PDE 2017-02-08, PDE 2017-02-22, PDE 2017-03-08, PDE 2017-03-22, PDE 2017-04-05, PDE 2017-04-19, Agent 2017-06-28, Agent 2017-07-12, Platform Core  Grooming  2017-08-22

Karen Van der Veer (JIRA)

unread,
Aug 24, 2017, 1:37:08 PM8/24/17
to puppe...@googlegroups.com
Karen Van der Veer updated an issue
Change By: Karen Van der Veer
Sprint: PDE 2017-02-08, PDE 2017-02-22, PDE 2017-03-08, PDE 2017-03-22, PDE 2017-04-05, PDE 2017-04-19,  Platform Core 2017-09-05,  Agent 2017-06-28, Agent 2017-07-12, Platform Core 2017-08-22

Moses Mendoza (JIRA)

unread,
Aug 24, 2017, 1:53:02 PM8/24/17
to puppe...@googlegroups.com
Moses Mendoza updated an issue
Change By: Moses Mendoza
Sprint: PDE 2017-02-08, PDE 2017-02-22, PDE 2017-03-08, PDE 2017-03-22, PDE 2017-04-05, PDE 2017-04-19,  Platform Core 2017-09-05,  Agent 2017-06-28, Agent 2017-07-12, Platform Core 2017-08-22

Kenn Hussey (JIRA)

unread,
Sep 4, 2017, 7:44:02 PM9/4/17
to puppe...@googlegroups.com
Kenn Hussey updated an issue
Change By: Kenn Hussey
Flagged: Impediment

Karen Van der Veer (JIRA)

unread,
Sep 5, 2017, 11:46:03 AM9/5/17
to puppe...@googlegroups.com
Karen Van der Veer updated an issue
Change By: Karen Van der Veer
Fix Version/s: PUP 5.2.0
Fix Version/s: PUP 5.y

Karen Van der Veer (JIRA)

unread,
Sep 5, 2017, 11:52:03 AM9/5/17
to puppe...@googlegroups.com
Karen Van der Veer updated an issue
Change By: Karen Van der Veer
Sprint: PDE 2017-02-08, PDE 2017-02-22, PDE 2017-03-08, PDE 2017-03-22, PDE 2017-04-05, PDE 2017-04-19, Agent 2017-06-28, Agent 2017-07-12, Platform Core  Grooming, Platform Core  2017-08-22

Karen Van der Veer (JIRA)

unread,
Sep 5, 2017, 2:35:03 PM9/5/17
to puppe...@googlegroups.com
Karen Van der Veer assigned an issue to Josh Cooper
Change By: Karen Van der Veer
Assignee: Josh Cooper

Karen Van der Veer (JIRA)

unread,
Sep 5, 2017, 3:17:04 PM9/5/17
to puppe...@googlegroups.com
Karen Van der Veer updated an issue
Change By: Karen Van der Veer
Fix Version/s: PUP 5.y
Fix Version/s: PUP 5.2.1

Kenn Hussey (JIRA)

unread,
Sep 6, 2017, 4:12:04 PM9/6/17
to puppe...@googlegroups.com
Kenn Hussey updated an issue
Change By: Kenn Hussey
Flagged: Impediment

Kenn Hussey (JIRA)

unread,
Sep 19, 2017, 10:48:02 AM9/19/17
to puppe...@googlegroups.com

Karen Van der Veer (JIRA)

unread,
Sep 19, 2017, 11:49:03 AM9/19/17
to puppe...@googlegroups.com
Karen Van der Veer updated an issue
Change By: Karen Van der Veer
Fix Version/s: PUP 5.3.0
Fix Version/s: PUP 5.3.z

Karen Van der Veer (JIRA)

unread,
Sep 19, 2017, 11:49:04 AM9/19/17
to puppe...@googlegroups.com
Karen Van der Veer updated an issue
Change By: Karen Van der Veer
Flagged: Impediment

Moses Mendoza (JIRA)

unread,
Oct 18, 2017, 8:05:04 PM10/18/17
to puppe...@googlegroups.com

Moses Mendoza (JIRA)

unread,
Oct 18, 2017, 8:05:04 PM10/18/17
to puppe...@googlegroups.com
Moses Mendoza assigned an issue to Unassigned
 
Change By: Moses Mendoza
Assignee: Josh Cooper

Moses Mendoza (JIRA)

unread,
Oct 24, 2017, 6:57:04 PM10/24/17
to puppe...@googlegroups.com
Moses Mendoza commented on Bug PUP-7102

Moses Mendoza (JIRA)

unread,
Oct 24, 2017, 6:57:04 PM10/24/17
to puppe...@googlegroups.com
Moses Mendoza assigned an issue to Thomas Hallgren
 
Change By: Moses Mendoza
Assignee: Thomas Hallgren

Thomas Hallgren (JIRA)

unread,
Oct 26, 2017, 6:21:03 AM10/26/17
to puppe...@googlegroups.com
Thomas Hallgren commented on Bug PUP-7102
 
Re: Insufficient logging when encountering 'permission denied' on environmentpath

Moses Mendoza, it's not obvious to me how that PR is related to the failures in the provided link. What should I look for, and where?

Thomas Hallgren (JIRA)

unread,
Oct 26, 2017, 7:13:03 AM10/26/17
to puppe...@googlegroups.com
Thomas Hallgren assigned an issue to Moses Mendoza
 
Change By: Thomas Hallgren
Assignee: Thomas Hallgren Moses Mendoza

Henrik Lindberg (JIRA)

unread,
Oct 26, 2017, 10:40:03 AM10/26/17
to puppe...@googlegroups.com
Henrik Lindberg commented on Bug PUP-7102
 
Re: Insufficient logging when encountering 'permission denied' on environmentpath

My guess is that there are acceptance tests that are wrong.

Craig Gomes (JIRA)

unread,
Oct 27, 2017, 4:36:09 PM10/27/17
to puppe...@googlegroups.com
Craig Gomes assigned an issue to Unassigned
 
Change By: Craig Gomes
Assignee: Moses Mendoza

Josh Cooper (JIRA)

unread,
Oct 30, 2017, 4:35:03 PM10/30/17
to puppe...@googlegroups.com
Josh Cooper assigned an issue to Thomas Hallgren
Change By: Josh Cooper
Assignee: Thomas Hallgren
This message was sent by Atlassian JIRA (v7.0.2#70111-sha1:88534db)
Atlassian logo

Thomas Hallgren (JIRA)

unread,
Oct 30, 2017, 5:07:16 PM10/30/17
to puppe...@googlegroups.com
Thomas Hallgren assigned an issue to Josh Cooper
Change By: Thomas Hallgren
Assignee: Thomas Hallgren Josh Cooper

Thomas Hallgren (JIRA)

unread,
Oct 30, 2017, 5:08:14 PM10/30/17
to puppe...@googlegroups.com
Thomas Hallgren commented on Bug PUP-7102
 
Re: Insufficient logging when encountering 'permission denied' on environmentpath

Josh Cooper, question arises, is the fix in this ticket at fault or should the acceptance tests change? I was under the impression that the acceptance tests actually did change at some point (the PR was reverted for that reason once before and then merged in again). If the acceptance tests are at fault, is there a ticket for that?

Josh Cooper (JIRA)

unread,
Dec 1, 2017, 3:09:03 PM12/1/17
to puppe...@googlegroups.com
Josh Cooper commented on Bug PUP-7102

The acceptance tests currently rely on the ability to start puppetserver and later adjust the permissions for the environment directories. We need to retarget the fix and update the existing acceptance tests so that they set the permissions correctly before starting puppetserver.

Josh Cooper (JIRA)

unread,
Dec 1, 2017, 3:09:04 PM12/1/17
to puppe...@googlegroups.com
Josh Cooper assigned an issue to Unassigned
 
Change By: Josh Cooper
Assignee: Josh Cooper

Josh Cooper (JIRA)

unread,
Mar 15, 2018, 8:17:03 PM3/15/18
to puppe...@googlegroups.com
Josh Cooper updated an issue
Change By: Josh Cooper
Sub-team: Language Coremunity
This message was sent by Atlassian JIRA (v7.7.1#77002-sha1:e75ca93)
Atlassian logo

Josh Cooper (JIRA)

unread,
Apr 3, 2018, 2:46:03 PM4/3/18
to puppe...@googlegroups.com
Josh Cooper updated an issue
Change By: Josh Cooper
Fix Version/s: PUP 5.3.6
Fix Version/s: PUP 5.3.z

Josh Cooper (JIRA)

unread,
Apr 3, 2018, 2:49:03 PM4/3/18
to puppe...@googlegroups.com
Josh Cooper commented on Bug PUP-7102
 
Re: Insufficient logging when encountering 'permission denied' on environmentpath

I bumped this out of 5.3.6, can land in 5.3.z at any time

Josh Cooper (JIRA)

unread,
Sep 26, 2018, 8:54:12 PM9/26/18
to puppe...@googlegroups.com
Josh Cooper updated an issue
Change By: Josh Cooper
Sprint: PDE 2017-02-08, PDE 2017-02-22, PDE 2017-03-08, PDE 2017-03-22, PDE 2017-04-05, PDE 2017-04-19, Agent 2017-06-28, Agent 2017-07-12, Platform Core Grooming, Platform Core 2017-08-22

Josh Cooper (JIRA)

unread,
Feb 27, 2019, 1:03:03 AM2/27/19
to puppe...@googlegroups.com
Josh Cooper updated an issue
Change By: Josh Cooper
Fix Version/s: PUP 5.3.z
Fix Version/s: PUP 5.5.z

Michiel van de Garde (Jira)

unread,
Feb 28, 2020, 7:20:03 AM2/28/20
to puppe...@googlegroups.com
Michiel van de Garde commented on Bug PUP-7102
 
Re: Insufficient logging when encountering 'permission denied' on environmentpath

Hey all! Could this issue be picked up again?

I noticed today in the puppet slack channel that there are still people getting confused and having to spend quite some time debugging this issue until they figure out what's happening.
This message was sent by Atlassian Jira (v8.5.2#805002-sha1:a66f935)
Atlassian logo

Josh Cooper (Jira)

unread,
Oct 23, 2020, 7:58:03 PM10/23/20
to puppe...@googlegroups.com
Josh Cooper updated an issue
Change By: Josh Cooper
Fix Version/s: PUP 5.5.z
Fix Version/s: PUP 6.y

Josh Cooper (Jira)

unread,
Feb 8, 2022, 2:45:01 AM2/8/22
to puppe...@googlegroups.com
Josh Cooper updated an issue
Change By: Josh Cooper
Team: Coremunity
This message was sent by Atlassian Jira (v8.20.2#820002-sha1:829506d)
Atlassian logo
Reply all
Reply to author
Forward
0 new messages