Commit Graph

27 Commits

Author SHA1 Message Date
Wyatt Walter
f448648783 Revert "chore: use bitnamilegacy images temporarily"
This reverts commit 64fc0a8141.
2025-08-11 15:16:23 -05:00
Wyatt Walter
64fc0a8141 chore: use bitnamilegacy images temporarily 2025-08-11 15:13:20 -05:00
Wyatt Walter
4c7a019adf
fix: support nss wrapper in Helm chart (#40673)
## Description

Requires: https://github.com/appsmithorg/appsmith/pull/40642

Adds an environment variable activating nss_wrapper when the
`securityPolicy.runAsUser` value is set so the UID can be set
dynamically. This avoids the `I have no name!` in the prompt when doing
a `kubectl exec` with that value set.

I am also introducing
[helm-unittest](https://github.com/helm-unittest/helm-unittest) for
ensuring that changes to our defaults are made explicit.

Fixes https://github.com/appsmithorg/appsmith/issues/38787

## Automation

/ok-to-test tags=""

### 🔍 Cypress test results
<!-- This is an auto-generated comment: Cypress test results  -->
> [!WARNING]
> Tests have not run on the HEAD
d30d87ffc66c107f980a3b27464e97db0910dcbe yet
> <hr>Mon, 19 May 2025 18:59:10 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**
- Added automated unit testing for Helm charts, including snapshot and
security context tests.
- Introduced documentation for running and understanding Helm chart unit
tests.
- Added a GitHub Actions workflow to run Helm chart unit tests on pull
requests and manually.
- **Bug Fixes**
- Ensured the LD_PRELOAD environment variable is set when a specific
security context is configured in deployments.
- **Documentation**
- Updated Helm chart README to reference new testing documentation and
improve clarity.
<!-- end of auto-generated comment: release notes by coderabbit.ai -->

---------

Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
Co-authored-by: Goutham Pratapa <goutham@appsmith.com>
2025-06-04 20:54:46 +05:30
Goutham Pratapa
de693f1b55
fix: security update init-container for redis (#39150)
<!-- This is an auto-generated comment: release notes by coderabbit.ai
-->

## Summary by CodeRabbit

- **Chores**
  - Updated the application release version to 3.6.2.
- Upgraded the Redis service container to a newer image version for
improved deployment consistency.

<!-- end of auto-generated comment: release notes by coderabbit.ai -->
2025-02-10 14:36:37 +05:30
Wyatt Walter
4caaa7411a
feat: allow persistent volume name to be overridden in the helm chart (#38752)
## Description

Exposes a value under the `persistence` key to allow the PV name to be
overridden. This was requested by a customer that does multiple deploys
of Appsmith per cluster in different namespaces with the same Helm
release name. Since PV's are not namespaced, the deploy was failing.

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  -->
> [!WARNING]
> Tests have not run on the HEAD
8a0e2ba2354c9a6742ca5d1a31d87559408325ec yet
> <hr>Thu, 23 Jan 2025 14:35:55 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

- **New Features**
	- Added ability to override Persistent Volume (PV) name in Helm chart.
	- Enhanced configuration options for multi-namespace deployments.

- **Improvements**
- Introduced more flexible naming strategy for Persistent Volume
resources.
- Updated PersistentVolume and PersistentVolumeClaim resources to
utilize the new naming convention.
	- Updated application version from 3.6.0 to 3.6.1.
<!-- end of auto-generated comment: release notes by coderabbit.ai -->
2025-01-29 13:49:59 +05:30
Goutham Pratapa
8cd827754f
chore: deprecate ce helm charts and publish ee helm charts (#37794)
<!-- This is an auto-generated comment: release notes by coderabbit.ai
-->
## Summary by CodeRabbit

## Release Notes

- **New Features**
- Enhanced support for PostgreSQL and OpenID Connect (OIDC)
authentication in the Helm chart.
- Introduced a new template for managing external secrets and a
dedicated service for metrics.
- Added Horizontal Pod Autoscaler (HPA) and Pod Disruption Budget (PDB)
configurations.
- New configuration options for custom Certificate Authority (CA)
certificates.

- **Improvements**
- Updated application version and dependencies for better organization
and readability.
- Expanded configuration options for Redis, MongoDB, and PostgreSQL,
allowing for improved customization.
	- Enhanced deployment configuration with dynamic scaling capabilities.

- **Bug Fixes**
- Improved error handling for configuration misalignments to prevent
runtime issues.

- **Documentation**
- Updated README and values.yaml to reflect new configuration parameters
and options.
<!-- end of auto-generated comment: release notes by coderabbit.ai -->

<!-- This is an auto-generated comment: Cypress test results  -->
> [!WARNING]
> Tests have not run on the HEAD
0e76b6af34501ed646ea840af22786b33426c9fe yet
> <hr>Tue, 31 Dec 2024 12:31:29 UTC
<!-- end of auto-generated comment: Cypress test results  -->
2024-12-31 18:01:48 +05:30
Goutham Pratapa
dda0290885
chore: update mongodb image (#37596)
<!-- This is an auto-generated comment: release notes by coderabbit.ai
-->
## Summary by CodeRabbit

- **New Features**
	- Updated MongoDB version to enhance performance and security.
- **Chores**
- Incremented application version from 2.2.1 to 2.3.0 for the Helm
chart.
	- Updated MongoDB image tag to version 6.0.13 for deployment.
<!-- end of auto-generated comment: release notes by coderabbit.ai -->

<!-- This is an auto-generated comment: Cypress test results  -->
> [!WARNING]
> Tests have not run on the HEAD
b0482e5de3e56bab523f901e3d2e7db481233216 yet
> <hr>Tue, 26 Nov 2024 09:59:51 UTC
<!-- end of auto-generated comment: Cypress test results  -->
2024-11-26 16:54:35 +05:30
Damien Léger
72670ae114
feat: allow HTTP port customization in Helm chart (#36763)
## Description
See https://github.com/appsmithorg/appsmith/issues/36761

I've been tested this with ee Helm chart. Imho tcp/8080 should be the
default port but to avoid breaking change, tcp/80 remains the default
port if the new variable is not specified.

Fixes #36761

## 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

- **New Features**
- Enhanced configurability of the HTTP container port through the
introduction of a new variable (commented out).
	- Updated the Helm chart version from 2.1.0 to 2.2.0.

- **Bug Fixes**
	- Improved indentation and formatting for better readability.

- **Documentation**
- Commented-out configuration for `HTTPContainerPort` included for
future reference.
<!-- end of auto-generated comment: release notes by coderabbit.ai -->

---------

Co-authored-by: Damien LEGER <damien.leger@cure51.com>
Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
Co-authored-by: Goutham Pratapa <pratapagoutham@gmail.com>
Co-authored-by: Goutham Pratapa <goutham@appsmith.com>
2024-11-01 15:16:00 +05:30
Goutham Pratapa
502d21b274
fix: fix appsmith-ce helm charts (#37137) 2024-10-30 16:36:19 +05:30
Shrikant Sharat Kandula
3fd1d52990
chore: Support APPSMITH_DB_URL in Helm chart (#33656)
If the `APPSMITH_MONGODB_URI` is not set, we set it to a constructed
service URL value. This PR adds the same feature to new
`APPSMITH_DB_URL` env variable as well.

⚠️ This will cause conflicts on sync. DO NOT MERGE unless the author is
available.

/test sanity

---------

Co-authored-by: Goutham Pratapa <goutham@appsmith.com>
2024-05-28 19:40:36 +05:30
Shrikant Sharat Kandula
8eea00146f
fix: Remove duplicate securityContext key (#28268)
Part of fix for #28242.
2023-10-24 13:37:05 +05:30
Goutham Pratapa
7728f9bf75
Fix images with vulnerabilities (#27733)
Fixes: [27523](https://github.com/appsmithorg/appsmith/issues/27523)
2023-10-12 11:20:24 +05:30
Sumesh Pradhan
dfff7c4a9f
chore: disable allowprivilegeescalation in the securitycontext of helm chart (#27041)
Updated` appsmith/appsmith` helm-chart to` 2.0.5` with
`allowPrivilegeEscalation` disabled.
2023-09-12 11:18:03 +05:30
Goutham Pratapa
e183d870bf
update: disable postgres by default in helm (#26614)
Co-authored-by: Shrikant Sharat Kandula <shrikant@appsmith.com>
2023-09-04 16:55:37 +05:30
Goutham Pratapa
82c5c0b580
chore: update init-container images and update probes (#23401)
Fixes: [21271](https://github.com/appsmithorg/appsmith/issues/21272)
Fixes: [221](https://github.com/appsmithorg/cloud-deployment/issues/221)
Fixes: [24020](https://github.com/appsmithorg/appsmith/issues/24020)
2023-07-07 10:01:43 +05:30
Goutham Pratapa
44d1837678
ci: disable postgres by default in helm charts (#23155) 2023-05-15 13:13:10 +05:30
Goutham Pratapa
b5b9cbd760
helm: add node selectors for appsmith sub-charts (#22195)
Fixes: [21796](https://github.com/appsmithorg/appsmith/issues/21796)
2023-05-08 10:00:11 +05:30
Goutham Pratapa
fa800a7704
fix: update chart.lock and app versions (#20105)
Fixes: [20000](https://github.com/appsmithorg/appsmith/issues/20000)

- [Bug]: Update appsmith version from 1.16.0 to the current 1.9.4
version
* Though we deploy the latest appsmith version. In the helm chart we
show that we are deploying appsmith version as 1.16.0 and its confusing
. We are now fixing it with this PR
- Correct helm chart versions for redis and mongo. This, helps in
deploying the desired versions of redis and mongo which appsmith
supports.
* Previously, When we deploy appsmith along with redis and mongo . It
used to deploy redis v7.0.4 and mongo v6.0.3.
with this changes we deploy the supported versions of redis and mongo
which is v6 and v5 respectively.

### Mongo

before:
```
I have no name!@tempappsmith-mongodb-0:/$ mongosh

Current Mongosh Log ID:	63d2128c2e4dabde5a68f8f8
Connecting to:		mongodb://127.0.0.1:27017/?directConnection=true&serverSelectionTimeoutMS=2000&appName=mongosh+1.6.0
Using MongoDB:		6.0.3
Using Mongosh:		1.6.0

For mongosh info see: https://docs.mongodb.com/mongodb-shell/

rs0 [direct: primary] test>

rs0 [direct: primary] test> db.version()
6.0.3
```

After the proposed changes:
```
I have no name!@tempappsmith-mongodb-0:/$ mongo
MongoDB shell version v5.0.9
connecting to: mongodb://127.0.0.1:27017/?compressors=disabled&gssapiServiceName=mongodb
Implicit session: session { "id" : UUID("d75e90e4-cbff-4b52-a7ab-22755ab43ec1") }
MongoDB server version: 5.0.9
================
Warning: the "mongo" shell has been superseded by "mongosh",
which delivers improved usability and compatibility.The "mongo" shell has been deprecated and will be removed in
an upcoming release.
For installation instructions, see
https://docs.mongodb.com/mongodb-shell/install/
================
rs0:PRIMARY> db.version()
5.0.9
```

### Redis

Before
```
I have no name!@tempappsmith-redis-master-0:/$ redis-server
19:C 26 Jan 2023 05:40:42.232 # oO0OoO0OoO0Oo Redis is starting oO0OoO0OoO0Oo
19:C 26 Jan 2023 05:40:42.232 # Redis version=7.0.4, bits=64, commit=00000000, modified=0, pid=19, just started
19:C 26 Jan 2023 05:40:42.232 # Warning: no config file specified, using the default config. In order to specify a config file use redis-server /path/to/redis.conf
19:M 26 Jan 2023 05:40:42.233 * monotonic clock: POSIX clock_gettime
19:M 26 Jan 2023 05:40:42.233 # Warning: Could not create server TCP listening socket *:6379: bind: Address already in use
19:M 26 Jan 2023 05:40:42.233 # Failed listening on port 6379 (TCP), aborting.
```

After the proposed changes
```
I have no name!@tempappsmith-redis-master-0:/$ redis-server
4728:C 26 Jan 2023 05:35:07.745 # oO0OoO0OoO0Oo Redis is starting oO0OoO0OoO0Oo
4728:C 26 Jan 2023 05:35:07.745 # Redis version=6.2.7, bits=64, commit=00000000, modified=0, pid=4728, just started
4728:C 26 Jan 2023 05:35:07.745 # Warning: no config file specified, using the default config. In order to specify a config file use redis-server /path/to/redis.conf
4728:M 26 Jan 2023 05:35:07.746 * monotonic clock: POSIX clock_gettime
4728:M 26 Jan 2023 05:35:07.746 # Warning: Could not create server TCP listening socket *:6379: bind: Address already in use
4728:M 26 Jan 2023 05:35:07.746 # Failed listening on port 6379 (TCP), aborting.
```
2023-02-01 17:12:00 +05:30
Sumesh Pradhan
c93ed4f54c
fix: helm install values checks (#19296)
## Description
- Updated ngnix as default ingress class in `values.yaml`
- Upgraded mongo's deployment image to `v5.0.12` as the previous 4.4.11
is deprecated
- Added `imagePullSecrets` to enable pulling images from private docker
registries
- Added conditional to check for k8s version >=1.1.8 to handle both old
and new k8s version scenarios for setting up the Ingress controller
class name.

Co-authored-by: Goutham Pratapa <goutham@appsmith.com>
2023-01-17 13:01:28 +05:30
Goutham Pratapa
97a3f2a30c
fix: Add mongo/redis to helm chart dependencies (#16096) 2022-10-14 16:15:49 +05:30
geekup-legodevops
95776e6454
Remove unnecessary variables in Helm chart & refactor templates (#12011) 2022-03-24 13:09:27 +05:30
geekup-legodevops
1e5c4651eb
Update Helm ingress template & document (#11552) 2022-03-09 12:07:19 +05:30
Shrikant Sharat Kandula
0a7ab37fba
Increment Helm charts version 2022-02-21 08:33:10 +05:30
geekup-legodevops
e85b34d66e
Add Helm configMap to update application configuration (#10157) 2022-01-31 13:27:01 +05:30
geekup-legodevops
269e3a786d
chore: Minor improvement for Helm, remove dependency & namespace (#9928) 2021-12-28 12:50:37 +05:30
gokuatkai
0edd50f6b2
Add helm param to disable namespace creation (#9719)
Will only create namespace if namespace.create == true
2021-12-15 08:01:48 +05:30
geekup-legodevops
ae26d2c611
Helm charts for Appsmith fat container (#7343) 2021-10-02 11:46:07 +05:30