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

(FACT-2802) Fix Cloud resolver #2093

Merged
merged 1 commit into from
Sep 21, 2020

Conversation

Filipovici-Andrei
Copy link
Contributor

When calling the cloud fact using a non root user,
an error is logged, because we attempt to access a privileged folder.

The fix skips folders which don't have a read permission.

@Filipovici-Andrei Filipovici-Andrei added the bug Something isn't working label Sep 18, 2020
@Filipovici-Andrei Filipovici-Andrei requested review from a team September 18, 2020 12:52
@Filipovici-Andrei
Copy link
Contributor Author

jenkins please test this on all

@gimmyxd
Copy link
Contributor

gimmyxd commented Sep 18, 2020

This should have a ticket associated, that way we can track the changes.
Even more if this is a bug that was introduced in a previously released facter.

@Filipovici-Andrei
Copy link
Contributor Author

jenkins please test this on all

@Filipovici-Andrei
Copy link
Contributor Author

@gimmyxd I'll create a ticket for this bug

@puppetcla
Copy link

CLA signed by all contributors.

@Filipovici-Andrei Filipovici-Andrei changed the title (maint) Fix Cloud resolver (FACT-2802) Fix Cloud resolver Sep 18, 2020
@oanatmaria oanatmaria merged commit 006e9e5 into puppetlabs:main Sep 21, 2020
@Filipovici-Andrei Filipovici-Andrei deleted the fix_cloud_fact branch September 29, 2020 15:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants