chore: 02/03/26 - Daily Promotion#41585
Merged
shadabbuchh merged 11 commits intomasterfrom Mar 2, 2026
Merged
Conversation
) ## Summary - Adds `extraVolumes` and `extraVolumeMounts` to the deployment parameters table in the Helm chart README - Adds commented usage examples (emptyDir mounted at `/tmp`) in `values.yaml` for both parameters Follow-up to #41515, which introduced the parameters but didn't include documentation. ## Test plan - [x] Verify `values.yaml` comments render correctly and the example is valid YAML when uncommented - [x] Verify README table renders correctly on GitHub 🤖 Generated with [Claude Code](https://claude.com/claude-code) <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit * **Documentation** * Added two new Helm chart parameters for configuring additional pod volumes and mounts. * Included example configurations to guide users in custom volume setup. <!-- end of auto-generated comment: release notes by coderabbit.ai --> Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
The `annotations` value (for adding annotations to the Deployment/StatefulSet resource) was added to values.yaml but was missing from the README parameters table. Added it to the deployment parameters section alongside `podAnnotations`. https://claude.ai/code/session_013oSUp3viPsj7rPsgXjLht5 <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit ## Release Notes * **Documentation** * Added new deployment configuration parameter for custom annotations on Deployment and StatefulSet resources, enabling users to add custom metadata during deployment configurations. <!-- end of auto-generated comment: release notes by coderabbit.ai --> Co-authored-by: Claude <noreply@anthropic.com>
## Description > [!TIP] > _Add a TL;DR when the description is longer than 500 words or extremely technical (helps the content, marketing, and DevRel team)._ > > _Please also include relevant motivation and context. List any dependencies that are required for this change. Add links to Notion, Figma or any other documents that might be relevant to the PR._ Fixes https://linear.app/appsmith/issue/APP-14832/support-ssh-keys-for-git-authentication-in-appsmith > [!WARNING] > _If no issue exists, please create an issue first, and check with the maintainers if the issue is valid._ ## Automation /ok-to-test tags="@tag.Git" ### 🔍 Cypress test results <!-- This is an auto-generated comment: Cypress test results --> > [!TIP] > 🟢 🟢 🟢 All cypress tests have passed! 🎉 🎉 🎉 > Workflow run: <https://github.com/appsmithorg/appsmith/actions/runs/22309832980> > Commit: 49c032c > <a href="https://internal.appsmith.com/app/cypress-dashboard/rundetails-65890b3c81d7400d08fa9ee5?branch=master&workflowId=22309832980&attempt=1" target="_blank">Cypress dashboard</a>. > Tags: `@tag.Git` > Spec: > <hr>Mon, 23 Feb 2026 15:11:26 UTC <!-- end of auto-generated comment: Cypress test results --> ## Communication Should the DevRel and Marketing teams inform users about this change? - [ ] Yes - [ ] No <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit - New Features - Connect and Import modals now support using existing SSH keys or generating new ones, with key fetching, selection, and inclusion in submissions. - Integrated SSH Key Manager (behind a feature flag) and exposed SSH key data in Git flows. - Added context provider to enable SSH key workflows in the import experience. - Style - Updated UI copy for Git connection steps (e.g., “Enter Repository URL”, “Set Up SSH Key”, “Configure SSH Key”, and refined SSH URL instructions). - Tests - Adjusted tests to match updated UI text. <!-- end of auto-generated comment: release notes by coderabbit.ai -->
…to TableWidgetV2 (#41551) Add widget-level row color style properties that allow users to customize header, odd, and even row background colors via the Style tab color pickers. Colors are applied via CSS classes on the TableWrapper styled component, with correct priority below selected-row/hover/cell-level overrides. ## Description Alternate row colors on tables Fixes #`8923` _or_ Fixes `Issue URL` > [!WARNING] > _If no issue exists, please create an issue first, and check with the maintainers if the issue is valid._ ## Automation /ok-to-test tags="@tag.All" ### 🔍 Cypress test results <!-- This is an auto-generated comment: Cypress test results --> > [!TIP] > 🟢 🟢 🟢 All cypress tests have passed! 🎉 🎉 🎉 > Workflow run: <https://github.com/appsmithorg/appsmith/actions/runs/22078556344> > Commit: 5f7961b > <a href="https://internal.appsmith.com/app/cypress-dashboard/rundetails-65890b3c81d7400d08fa9ee5?branch=master&workflowId=22078556344&attempt=1" target="_blank">Cypress dashboard</a>. > Tags: `@tag.All` > Spec: > <hr>Mon, 16 Feb 2026 23:08:48 UTC <!-- end of auto-generated comment: Cypress test results --> ## Communication Should the DevRel and Marketing teams inform users about this change? - [ ] Yes - [X] No <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit ## Release Notes * **New Features** * Table widgets now support custom row and header colors. Users can configure header row background, header text, odd row background, and even row background colors for enhanced visual customization. * **Refactor** * Optimized table row styling logic and removed unused code expressions. <!-- end of auto-generated comment: release notes by coderabbit.ai --> --------- Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com> Co-authored-by: subrata71 <subrata@appsmith.com> Co-authored-by: Cursor <cursoragent@cursor.com>
## Description Feature Request 8479: Allow a user to favorite applications. In bigger factories there can be many workspaces and applications. Users might have access to a lot of applications, but generally only use a handful. This feature allows a user to favorite one or more applications (up to 50). Any favorited applications will show up in a virtual Favorites workspace in alphabetical order Fixes #`8479` ## Automation /ok-to-test tags="@tag.All" ### 🔍 Cypress test results <!-- This is an auto-generated comment: Cypress test results --> > [!TIP] > 🟢 🟢 🟢 All cypress tests have passed! 🎉 🎉 🎉 > Workflow run: <https://github.com/appsmithorg/appsmith/actions/runs/22001834951> > Commit: 56d760c > <a href="https://internal.appsmith.com/app/cypress-dashboard/rundetails-65890b3c81d7400d08fa9ee5?branch=master&workflowId=22001834951&attempt=1" target="_blank">Cypress dashboard</a>. > Tags: `@tag.All` > Spec: > <hr>Fri, 13 Feb 2026 21:58:17 UTC <!-- end of auto-generated comment: Cypress test results --> ## Communication Should the DevRel and Marketing teams inform users about this change? - [X] Yes - [ ] No <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit * **New Features** * Mark applications as favorites via a heart toggle on application cards (limit: 50) * Virtual "Favorites" workspace and dedicated view listing favorited applications * Endpoints and flows to fetch/sync favorite applications * **UX** * Favorites reflected across listings, headers, and workspace navigation (URL support) * Optimistic toggles with rollback on error and toast feedback; auto-refresh of favorites when workspaces change * **Chores** * Deleted applications are removed from users' favorites automatically <!-- end of auto-generated comment: release notes by coderabbit.ai --> --------- Co-authored-by: Cursor <cursoragent@cursor.com> Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
#41576) ## Description > [!TIP] > _Add a TL;DR when the description is longer than 500 words or extremely technical (helps the content, marketing, and DevRel team)._ > > _Please also include relevant motivation and context. List any dependencies that are required for this change. Add links to Notion, Figma or any other documents that might be relevant to the PR._ …support - Inject logs-capture.js and recorder.js in index.html when BetterBugs is enabled and not airgapped; respect APPSMITH_DISABLE_BETTERBUGS and APPSMITH_AIRGAP_ENABLED - Set __BetterbugsRecordingLinkConfig with default primaryColor #E15615 and success copy aligned with in-app widget - Declare DISABLE_BETTERBUGS once in head and remove duplicate in body to fix redeclaration error - Add JSDoc in betterbugs.ts noting recording-link scripts are loaded in index.html Fixes #`Issue Number` _or_ Fixes `Issue URL` > [!WARNING] > _If no issue exists, please create an issue first, and check with the maintainers if the issue is valid._ ## Automation /ok-to-test tags="@tag.All" ### 🔍 Cypress test results <!-- This is an auto-generated comment: Cypress test results --> > [!TIP] > 🟢 🟢 🟢 All cypress tests have passed! 🎉 🎉 🎉 > Workflow run: <https://github.com/appsmithorg/appsmith/actions/runs/22415681310> > Commit: 47e9bc5 > <a href="https://internal.appsmith.com/app/cypress-dashboard/rundetails-65890b3c81d7400d08fa9ee5?branch=master&workflowId=22415681310&attempt=2" target="_blank">Cypress dashboard</a>. > Tags: `@tag.All` > Spec: > <hr>Wed, 25 Feb 2026 22:45:26 UTC <!-- end of auto-generated comment: Cypress test results --> ## Communication Should the DevRel and Marketing teams inform users about this change? - [ ] Yes - [ ] No <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit * **Chores** * Updated BetterBugs recording script configuration to support conditional loading based on environment settings, improving configuration management and eliminating duplicate variable declarations. * **Documentation** * Added comprehensive documentation describing BetterBugs in-app recording functionality, including details on script loading behavior and when features activate. <!-- end of auto-generated comment: release notes by coderabbit.ai -->
Add `encode zstd gzip` to the Caddy configuration so that responses are compressed based on the client's Accept-Encoding header. This ensures customers get compressed API and static responses even when their ingress controller or load balancer doesn't handle compression (e.g., layer 4 proxies). ## Description > [!TIP] > _Add a TL;DR when the description is longer than 500 words or extremely technical (helps the content, marketing, and DevRel team)._ > > _Please also include relevant motivation and context. List any dependencies that are required for this change. Add links to Notion, Figma or any other documents that might be relevant to the PR._ Fixes #`Issue Number` _or_ Fixes `Issue URL` > [!WARNING] > _If no issue exists, please create an issue first, and check with the maintainers if the issue is valid._ ## Automation /ok-to-test tags="" ### 🔍 Cypress test results <!-- This is an auto-generated comment: Cypress test results --> > [!CAUTION] > If you modify the content in this section, you are likely to disrupt the CI result for your PR. <!-- end of auto-generated comment: Cypress test results --> ## Communication Should the DevRel and Marketing teams inform users about this change? - [ ] Yes - [ ] No <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit * **New Features** * Enabled HTTP response compression to automatically reduce content sizes for improved performance and faster downloads. * **Tests** * Added test coverage to verify on-the-fly compression behavior across different encoding types and client scenarios. <!-- end of auto-generated comment: release notes by coderabbit.ai --> Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
## Description > [!TIP] > _Add a TL;DR when the description is longer than 500 words or extremely technical (helps the content, marketing, and DevRel team)._ > > _Please also include relevant motivation and context. List any dependencies that are required for this change. Add links to Notion, Figma or any other documents that might be relevant to the PR._ It appears that we pinned the version of the JDK a long time ago because the Temurin binaries changed distribution source and no one brought this forward. This is the version of Java that our CI runners are using, so bringing the image up-to-date to address the mismatch. Fixes #`Issue Number` _or_ Fixes `Issue URL` > [!WARNING] > _If no issue exists, please create an issue first, and check with the maintainers if the issue is valid._ ## Automation /ok-to-test tags="" ### 🔍 Cypress test results <!-- This is an auto-generated comment: Cypress test results --> > [!CAUTION] > If you modify the content in this section, you are likely to disrupt the CI result for your PR. <!-- end of auto-generated comment: Cypress test results --> ## Communication Should the DevRel and Marketing teams inform users about this change? - [ ] Yes - [x] No <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit * **Chores** * Enhanced Docker image builds to support multiple processor architectures, enabling deployments across diverse system configurations. <!-- end of auto-generated comment: release notes by coderabbit.ai -->
…lowError (#41575) addCookieInitializer() was called on every setSessionId() invocation, permanently chaining Consumer.andThen() on the singleton bean's initializer field. After enough session-setting requests, the chain depth exceeded the JVM stack size, crashing SSO login with a StackOverflowError. Move the sameSite(Lax) initializer to the constructor so it's set once. Leave addCookieInitializers(exchange) as a no-op hook for EE subclasses. ## Description > [!TIP] > _Add a TL;DR when the description is longer than 500 words or extremely technical (helps the content, marketing, and DevRel team)._ > > _Please also include relevant motivation and context. List any dependencies that are required for this change. Add links to Notion, Figma or any other documents that might be relevant to the PR._ Fixes #`Issue Number` _or_ Fixes `Issue URL` > [!WARNING] > _If no issue exists, please create an issue first, and check with the maintainers if the issue is valid._ ## Automation /ok-to-test tags="" ### 🔍 Cypress test results <!-- This is an auto-generated comment: Cypress test results --> > [!CAUTION] > If you modify the content in this section, you are likely to disrupt the CI result for your PR. <!-- end of auto-generated comment: Cypress test results --> ## Communication Should the DevRel and Marketing teams inform users about this change? - [ ] Yes - [x] No <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit * **Bug Fixes** * Improved session cookie configuration stability by consolidating SameSite attribute setup and preventing potential initialization accumulation issues. <!-- end of auto-generated comment: release notes by coderabbit.ai --> Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
Contributor
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. 🗂️ Base branches to auto review (2)
Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
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 |
… restarts (#41584) ## Description [Shadow EE PR](appsmithorg/appsmith-ee#8617) # Delete Flow Diagnostics Insights ## What was added - Point-in-time diagnostics around app delete flow (`start` and `end-success/end-error`). - Captured: - JVM memory (`heap`, `non-heap`, memory pools like `G1 Eden`, `G1 Old Gen`) - Thread stats (`eventLoop`, `elastic`, `parallel`, `other`, total threads) - Reactive Mongo connection stats (`current`, `available`, `totalCreated`, `active`) - Elapsed delete duration and start/end deltas ## High-level observations from runs shared ### Earlier heavy runs (4-run batch) - Duration mostly high: ~7s to ~9.5s. - Memory behavior volatile: - 3 runs had positive heap growth (`+1.69 GiB`, `+0.89 GiB`, `+0.33 GiB`) - 1 run had heap drop (`-0.73 GiB`) - Largest spike showed strong `G1 Old Gen` growth (promotion/retention pressure). - Mongo connections were stable in most runs, but one run expanded pool (`current +16`, `available -16`). - Thread counts were broadly stable (no sustained thread leak pattern). ### Later stable runs (3-run batch) - Faster: ~1.3s to ~3.3s. - Heap mostly stable or improved: - `+17 MB`, `-28 MB`, `-13 MB` - Mongo mostly stable: - first run warmed up (`current +6`), then flat. - Threads mostly stable: - one run `+6` (eventLoop), then flat. ## What this indicates - Delete flow now shows better runtime behavior in the later runs: - lower latency - smaller/negative heap deltas - stable thread and DB connection behavior after warm-up - No consistent leak signature seen in the stable batch. ## Suggested way to report improvement - “We now have start/end diagnostics for delete flow covering memory, thread pools, and Mongo connections.” - “Compared to earlier runs, recent runs are faster and show controlled memory behavior.” - “Connection/thread growth appears warm-up related and stabilizes in subsequent runs.” ## Guardrails to monitor going forward - `elapsedMs` consistently > 5000 - `heapUsedDeltaBytes` repeatedly > `+512 MB` - repeated `G1 Old Gen` growth across consecutive deletes - `mongoCurrentConnectionsDelta` repeatedly > `+8` - monotonic rise in `totalThreadCount` across runs Fixes https://linear.app/appsmith/issue/APP-14877/appsmith-crashes-and-restarts-on-app-deletion-with-high-cpu-usage ## Automation /ok-to-test tags="@tag.All" ### 🔍 Cypress test results <!-- This is an auto-generated comment: Cypress test results --> > [!TIP] > 🟢 🟢 🟢 All cypress tests have passed! 🎉 🎉 🎉 > Workflow run: <https://github.com/appsmithorg/appsmith/actions/runs/22549602904> > Commit: 6ae8821 > <a href="https://internal.appsmith.com/app/cypress-dashboard/rundetails-65890b3c81d7400d08fa9ee5?branch=master&workflowId=22549602904&attempt=2" target="_blank">Cypress dashboard</a>. > Tags: `@tag.All` > Spec: > <hr>Mon, 02 Mar 2026 04:51:49 UTC <!-- end of auto-generated comment: Cypress test results --> ## Communication Should the DevRel and Marketing teams inform users about this change? - [ ] Yes - [ ] No <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit ## Release Notes * **Bug Fixes** * Optimized bulk archival operations for applications, action collections, actions, and pages * Enhanced concurrency controls in application deletion workflows * Improved analytics event handling to prevent deletion failures from analytics errors <!-- end of auto-generated comment: release notes by coderabbit.ai -->
yatinappsmith
approved these changes
Mar 2, 2026
This file contains hidden or 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
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
Tip
Add a TL;DR when the description is longer than 500 words or extremely technical (helps the content, marketing, and DevRel team).
Please also include relevant motivation and context. List any dependencies that are required for this change. Add links to Notion, Figma or any other documents that might be relevant to the PR.
Fixes #
Issue Numberor
Fixes
Issue URLWarning
If no issue exists, please create an issue first, and check with the maintainers if the issue is valid.
Automation
/ok-to-test tags=""
🔍 Cypress test results
Warning
Tests have not run on the HEAD be5727e yet
Mon, 02 Mar 2026 05:56:47 UTC
Communication
Should the DevRel and Marketing teams inform users about this change?