Skip to content

Commit

Permalink
chore: remove kafka-zookeeper image (#52)
Browse files Browse the repository at this point in the history
  • Loading branch information
ravisingal authored Oct 1, 2024
1 parent f52ed31 commit 479c75f
Show file tree
Hide file tree
Showing 10 changed files with 2 additions and 272 deletions.
44 changes: 0 additions & 44 deletions .github/workflows/docker-tests.yml

This file was deleted.

11 changes: 0 additions & 11 deletions .github/workflows/kafka-zookeeper/docker-compose.yml

This file was deleted.

9 changes: 1 addition & 8 deletions .github/workflows/pr-build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@ jobs:
- name: Build with Gradle
uses: hypertrace/github-actions/gradle@main
with:
args: build dockerBuildImages
args: dockerBuildImages

- name: Run Trivy vulnerability scanner for kafka image
uses: hypertrace/github-actions/trivy-image-scan@main
Expand All @@ -31,13 +31,6 @@ jobs:
output-mode: github
category: kafka

- name: Run Trivy vulnerability scanner for kafka-zookeeper image
uses: hypertrace/github-actions/trivy-image-scan@main
with:
image: hypertrace/kafka-zookeeper
output-mode: github
category: kafka-zookeeper

validate-helm-charts:
runs-on: ubuntu-20.04
steps:
Expand Down
14 changes: 1 addition & 13 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -22,25 +22,13 @@ To build kafka image locally, run:
```

`Note:`
- docker-compose uses `kafka-zookeeper` image so you have to build it from that folder in case you are working on that one.
- To read more about installing and configuring helm chart refer [BUILD.md](/BUILD.md).

## Testing

You can test the image you built after modification by running docker-compose or helm setup.

### docker-compose
Change the tag for `kafka-zookeeper` from `:main` to `:test` in [docker-compose file](https://github.com/hypertrace/hypertrace/blob/main/docker/docker-compose.yml) like this.

```yaml
kafka-zookeeper:
image: hypertrace/kafka-zookeeper:test
container_name: kafka-zookeeper
...
```
You can test the image you built after modification by running docker-compose or helm setup.

and then run `docker-compose up` to test the setup.

## Docker Image Source:
- [DockerHub > kafka](https://hub.docker.com/r/hypertrace/kafka)
- [DockerHub > kafka-zookeeper](https://hub.docker.com/r/hypertrace/kafka-zookeeper)
40 changes: 0 additions & 40 deletions kafka-zookeeper/Dockerfile

This file was deleted.

9 changes: 0 additions & 9 deletions kafka-zookeeper/build.gradle.kts

This file was deleted.

31 changes: 0 additions & 31 deletions kafka-zookeeper/docker-bin/docker-healthcheck

This file was deleted.

26 changes: 0 additions & 26 deletions kafka-zookeeper/docker-bin/start-kafka-zookeeper

This file was deleted.

89 changes: 0 additions & 89 deletions kafka-zookeeper/install.sh

This file was deleted.

1 change: 0 additions & 1 deletion settings.gradle.kts
Original file line number Diff line number Diff line change
Expand Up @@ -10,5 +10,4 @@ plugins {
id("org.hypertrace.version-settings") version "0.2.0"
}

include(":kafka-zookeeper")
include(":docker")

0 comments on commit 479c75f

Please sign in to comment.