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

Prefer MiqProductFeature cache invalidation explicitly via backend #7430

Merged
merged 1 commit into from
Nov 16, 2020

Conversation

skateman
Copy link
Member

@skateman skateman commented Oct 20, 2020

The cache invalidation should be done implicitly when saving a Tenant record...

Depends on ManageIQ/manageiq#20708 ManageIQ/manageiq#20772

@miq-bot
Copy link
Member

miq-bot commented Oct 20, 2020

Checked commit skateman@9ec2173 with ruby 2.6.3, rubocop 0.69.0, haml-lint 0.28.0, and yamllint
3 files checked, 0 offenses detected
Everything looks fine. 🍪

@gtanzillo gtanzillo assigned gtanzillo and h-kataria and unassigned gtanzillo Oct 22, 2020
kbrock added a commit to kbrock/manageiq that referenced this pull request Nov 2, 2020
miq product features contain tenant data. So we want to update the product features data
on all servers when a tenant changes

This allows us to remove special logic from the front end.

ManageIQ/manageiq-ui-classic#7430
kbrock added a commit to kbrock/manageiq that referenced this pull request Nov 6, 2020
miq product features contain tenant data. So we want to update the product features data
on all servers when a tenant changes

This allows us to remove special logic from the front end.

ManageIQ/manageiq-ui-classic#7430
kbrock added a commit to kbrock/manageiq that referenced this pull request Nov 12, 2020
miq product features contain tenant data. So we want to update the product features data
on all servers when a tenant changes

This allows us to remove special logic from the front end.

ManageIQ/manageiq-ui-classic#7430
@Fryguy Fryguy merged commit 0716d42 into ManageIQ:master Nov 16, 2020
@Fryguy Fryguy assigned Fryguy and unassigned h-kataria Nov 16, 2020
@skateman skateman deleted the invalidate-caches-cleanup branch November 16, 2020 15:29
mzazrivec pushed a commit to mzazrivec/manageiq that referenced this pull request Dec 10, 2020
miq product features contain tenant data. So we want to update the product features data
on all servers when a tenant changes

This allows us to remove special logic from the front end.

ManageIQ/manageiq-ui-classic#7430
(cherry picked from commit c7aa203)
mzazrivec pushed a commit to mzazrivec/manageiq that referenced this pull request Dec 18, 2020
miq product features contain tenant data. So we want to update the product features data
on all servers when a tenant changes

This allows us to remove special logic from the front end.

ManageIQ/manageiq-ui-classic#7430
(cherry picked from commit c7aa203)
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.

5 participants