-
Notifications
You must be signed in to change notification settings - Fork 111
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
[CON-230] - Bump nginx cache ttl #3323
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
vicky-g
changed the title
[CON-230] - Bump nginxx cache ttl
[CON-230] - Bump nginx cache ttl
Jun 27, 2022
SidSethi
approved these changes
Jun 27, 2022
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm! can you post in eng-content as fyi for draj/ray
SidSethi
pushed a commit
that referenced
this pull request
Jun 30, 2022
* bump cache ttl * set inactive to 12h too * add nginx comments * comments * comment update
SidSethi
pushed a commit
that referenced
this pull request
Jun 30, 2022
* bump cache ttl * set inactive to 12h too * add nginx comments * comments * comment update
SidSethi
pushed a commit
that referenced
this pull request
Jun 30, 2022
* bump cache ttl * set inactive to 12h too * add nginx comments * comments * comment update
sliptype
pushed a commit
that referenced
this pull request
Sep 10, 2023
sliptype
pushed a commit
that referenced
this pull request
Sep 10, 2023
[eaaf509] [C-2565] Improve mobile app-redirect-popover experience (#3344) Dylan Jeffers [1755765] Update sdk to 2.0.3-beta.18 (#3345) Sebastian Klingler [571c991] [CON-676] Support storage v2 playlists+albums, and bump libs version to 2.0.3-beta.17 (#3314) Theo Ilie [3cca778] [C-2577] Fix mobile web social buttons by removing pull-to-refresh (#3343) Dylan Jeffers [0915ff4] [C-2519] Ignore TOS deeplink (#3341) Dylan Jeffers [39fda1a] Feature flag gates to send metadata directly through chain (#3311) Michelle Brier [80278ce] Upgrade sdk to beta.16, update install script (#3339) Dylan Jeffers [7c3aaa5] [C-2451] Fix infinite buffering in audio player (#3336) Sebastian Klingler [be68f2e] Prevent fetchBlockees/blockers when chat disabled (#3338) Dylan Jeffers [779cfa5] [C-2568] Silence player/error when audio is loaded (#3330) Sebastian Klingler [869504c] HOTFIX track notification navigation (#3335) Dylan Jeffers [f30507c] Dismiss keyboard on mobile chat screen top right press (#3333) Reed [ad7938c] DMs: Don't increment unread if it's your own message (#3331) Marcus Pasell [565f182] Fix keyboard occluding textinput in mobile chat screen (#3332) Reed [a82cf59] DMs: Fix unread indicator not going away (#3329) Marcus Pasell [e47f323] Mobile chat housekeeping (#3288) Reed [47a5e84] [PAY-1207] Use Followers instead of Mutuals on compose chat modal (#3323) Marcus Pasell [03c587d] [C-2393] Cache pods in CI (#3317) Sebastian Klingler [22f4d46] Remove identity subscriptions reads/writes (#3005) Michelle Brier [b9ddae1] [C-2566] Prevent player error on skip (#3327) Dylan Jeffers [ca246f9] DMs: Fix store bug sending messages (#3326) Marcus Pasell [309af58] Update fetchPermission condition (#3324) Dylan Jeffers [a8e301e] [PAY-1199] Mobile inbox settings functional (#3290) Reed [adc7854] [C-2563] Fix demo build (#3322) Sebastian Klingler [d703f6e] Fix search-results saga error due to yield* (#3321) Dylan Jeffers [d940ae6] Fix internal/external audius urls (#3320) Dylan Jeffers [96cd42f] [C-2560] Improve checking for new notifs (#3316) Dylan Jeffers [3130546] I broke main and I'm sorry (#3319) Marcus Pasell [a3062fe] Remove dependency on `download.cid` for determining download eligibility (#3307) Michael Piazza [fe04046] [PAY-901] hotfix: Use SPL amounts for BuyAudio flow (#3318) Marcus Pasell [1c94f06] Fix mobile prem content track headers (#3315) Reed [f87cb04] [PAY-1108] Chats Screen avoids play bar (#3295) Reed [c5450a0] Revert "[PAY-901] hotfix: TransactionDetailsModal now expects wei amounts" (#3313) Marcus Pasell [e3ca9e3] Fix stylelint (#3312) Raymond Jacobson [2031f1a] Debounce reaction for longer on web (#3305) Raymond Jacobson [492fa67] Notification settings improvements (#3264) Michelle Brier [59710d6] Rename sdk_v2 to sdk_discovery_node_selector (#3310) Dylan Jeffers [8519910] [PLAT-863] Fix reaction endpoint when sdk-v2 (#3309) Dylan Jeffers [00ba722] Fix index issue with constructUrl (#3308) Dylan Jeffers [b0b32ff] [PLAT-904] Fix duplicate notifications (#3304) Dylan Jeffers [14c7814] Pass metadata to libs for addTracksToChainAndCnode (#3285) Michelle Brier [75456fd] Add search multimap (#3306) Raymond Jacobson [f8f5f02] [C-2282] Show tracks tab on own profile if all tracks hidden (#3294) Andrew Mendelsohn [49fc834] [PAY-901] hotfix: TransactionDetailsModal now expects wei amounts (#3297) Marcus Pasell [f1366ac] Fix eager fetch (#3302) Raymond Jacobson [4b639ba] [C-2542] Update delete playlist confirmation drawer to conform to new designs (#3303) Kyle Shanks [1ac5ec2] Fix taste maker copy (#3301) Isaac Solo [de48e11] [PAY-1178] Fix new messages on new/unloaded chats (#3298) Marcus Pasell [ea6ed55] Redirect if not allowed ai (#3300) Raymond Jacobson [985a05c] Fix ai attrib upload (#3299) Raymond Jacobson [a586435] [C-2554] Add AI Attribution (#3296) Dylan Jeffers [64dde7f] [CON-674] Make track edit use v2 flow for storage v2 tracks (#3289) Theo Ilie [d38d483] [C-2543, C-2544] Initial updates to mobile playlist management (#3287) Kyle Shanks [ec9dc0e] Fix devnet in wallet authorize for saga phone (#3291) Raymond Jacobson
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Might as well bump the cache ttl to 12h from 30m, since the cn5 cache enabled soak is working out well.
Tests
No change in functionality, just bumping the ttl
Monitoring - How will this change be monitored? Are there sufficient logs / alerts?
Can still check the nginx access.log and error.log
Also if
x-cache-status
is in the response headers, then the nginx cache is enabled