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 #10383: If I click on "Regenerate all policies", it clears the cache of changes, and slows down a lot the web interface #1576

Conversation

fanf
Copy link
Member

@fanf fanf commented Mar 10, 2017

@VinceMacBuche
Copy link
Member

You need to rebase your pull request

set
}) match {
case eb:EmptyBox =>
val e = eb ?~! "Error when logging the cache event"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The error may not be wile logging the cache event but also when clearNodeConfigurationCache fails

EventLogDetails(
modificationId = Some(modId)
, principal = CurrentUser.getActor
, details = EventLog.emptyDetails
, reason = None))) match {
, reason = Some("Node configuration caches deleted on user request")
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"cache" instead of "caches"

…e cache of changes, and slows down a lot the web interface
@fanf
Copy link
Member Author

fanf commented Mar 13, 2017

PR rebased

@fanf fanf force-pushed the bug_10383/if_i_click_on_regenerate_all_policies_it_clears_the_cache_of_changes_and_slows_down_a_lot_the_web_interface branch from 65eb9f1 to efac891 Compare March 13, 2017 08:40
@Normation-Quality-Assistant
Copy link
Contributor

OK, merging this PR

@Normation-Quality-Assistant Normation-Quality-Assistant merged commit efac891 into Normation:branches/rudder/4.1 Mar 13, 2017
@fanf fanf deleted the bug_10383/if_i_click_on_regenerate_all_policies_it_clears_the_cache_of_changes_and_slows_down_a_lot_the_web_interface branch March 15, 2024 10:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants