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

fixes #20638 - manifest import/refresh as anonymous user #6922

Merged
merged 1 commit into from
Aug 28, 2017

Conversation

bbuckingham
Copy link
Member

This change performs the manifest import/refresh as anonymous user.

Without this change, behavior seen was that doing a new import or
refresh did not result in any changes to Katello DB. E.g. Product
table was empty after import and no subscriptions could be seen.

While debugging, we found that the save of the products
was failing due to permissions error that resulted from not having
an assigned user.

This change performs the manifest import/refresh as anonymous user.

Without this change, behavior seen was that doing a new import or
refresh did not result in any changes to Katello DB.  E.g. Product
table was empty after import and no subscriptions could be seen.

While debugging, we found that the save of the products
was failing due to permissions error that resulted from not having
an assigned user.
@theforeman-bot
Copy link

Issues: #20638

@cfouant
Copy link
Member

cfouant commented Aug 25, 2017

ACK thanks @bbuckingham!

@ehelms
Copy link
Member

ehelms commented Aug 25, 2017

@bbuckingham Based on what the fix ended up being, I'm inclined to bring this into 3.4.5 as we've seen similar issues happen that required the same fix. I think that would help safeguard against it happening. If you agree, feel free to flip the issue to the 3.4.5 release.

@bbuckingham
Copy link
Member Author

@cfouant, Thanks!
@ehelms, Thanks! Updated the redmine target to 3.4.5. :)

@bbuckingham bbuckingham merged commit d561b9f into Katello:master Aug 28, 2017
@bbuckingham bbuckingham deleted the 20638 branch August 28, 2017 15:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants