add default cache-control headers for module web files - #6986
Conversation
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
@christianvogt did you have a 2.25 cluster to test it |
|
Created a custom image and replace dashboard image on test cluster. Sent you test cluster details. |
|
tested, looks good |
|
[APPROVALNOTIFIER] This PR is APPROVED Approval requirements bypassed by manually added approval. This pull-request has been approved by: The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
84a98b3
into
opendatahub-io:stable-2.x
fixes https://redhat.atlassian.net/browse/RHOAIENG-52182
Back port of #6632 for
cache-controlheaders.Back port a small change from #5080 for
moduleFederation.jsThis backport is to prevent cache related issues which occur after an upgrade.
Tested against 2.25.3 cluster with image build from local changes.
cache-control: no-cacheheader on requests made to/_mf/.