Skip to content

Commit

Permalink
chore: Remove unused netcat and openssh-client (#37662)
Browse files Browse the repository at this point in the history
These packages were installed in
daf60fa
for use with the `appsmithctl migrate` command, which has never been
documented, been deprecated and then deleted from code for some time
now.

Removing these packages now as we don't need them any longer.

Tested on EE and verified `/test all` to have passed.

## Automation

/test sanity

### 🔍 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**
	- Updated the base image to `ubuntu:20.04` for improved compatibility.
- Introduced a new builder stage for Caddy with the `caddy-ratelimit`
module.
- Enhanced support for internationalization by setting environment
variables for language and locale.

- **Improvements**
- Streamlined package installation process by removing unnecessary
packages.
- Optimized the final image size with cleanup commands and maintained
essential database services.

<!-- end of auto-generated comment: release notes by coderabbit.ai -->
  • Loading branch information
sharat87 authored Nov 23, 2024
1 parent a4d35d2 commit 5c15017
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 7 deletions.
7 changes: 1 addition & 6 deletions deploy/docker/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -159,12 +159,7 @@ This will need a restart of the Appsmith server, which can be done using the fol
```sh
docker-compose exec appsmith-ce supervisorctl restart backend
```
### Migrate To New Server
To migrate a running container to a new server or other machine, you can use the following `migrate` command and replace the `<user>` and `<new-server-ip-address>` by the user and IP address of the destination server or machine.
```
docker exec appsmith-ce appsmithctl migrate <user>@<new-server-ip-address>
```
This command will migrate all data and configuration of running container on source machine to destination machine and start a new container on destination machine.

## Supervisor

The container runs multiple processes, including the Appsmith server, Nginx, MongoDB etc., inside a single Docker container. These processes are started and managed by [supervisord](http://supervisord.org/).
Expand Down
2 changes: 1 addition & 1 deletion deploy/docker/base.dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@ RUN set -o xtrace \
&& apt-get update \
&& apt-get upgrade --yes \
&& DEBIAN_FRONTEND=noninteractive apt-get install --no-install-recommends --yes \
supervisor curl nfs-common gnupg wget netcat openssh-client \
supervisor curl nfs-common gnupg wget \
gettext \
ca-certificates \
# Install MongoDB v5, Redis, PostgreSQL v13
Expand Down

0 comments on commit 5c15017

Please sign in to comment.