Skip to content
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

Fix docker compose deployment #265

Merged
merged 3 commits into from
Sep 13, 2024
Merged

Conversation

jt-dd
Copy link
Contributor

@jt-dd jt-dd commented Sep 13, 2024

Fix issue regarding the docker compose stack deployement #257. Adding env variable setup example for easier setup in docker compose stack.

      # Custom config for docker compose environment
      - KH_MONGODB_URL=mongodb://mongodb:27017
      - KH_JANUSGRAPH_URL=ws://kubegraph:8182/gremlin
      # Default config
      - KH_INGESTOR_API_ENDPOINT=0.0.0.0:9000
      - KH_INGESTOR_TEMP_DIR=/tmp/kubehound
      - KH_INGESTOR_MAX_ARCHIVE_SIZE=2147483648 # 2GB
      - KH_INGESTOR_ARCHIVE_NAME=archive.tar.gz
      # AWS Bucket configuration
      - KH_INGESTOR_REGION=us-east-1
      - KH_INGESTOR_BUCKET_NAME= # s3://<your_bucket>
      - AWS_ACCESS_KEY_ID=
      - AWS_SECRET_ACCESS_KEY=
      - AWS_SESSION_TOKEN= # for aws-vault generated credentials

@jt-dd jt-dd requested a review from a team as a code owner September 13, 2024 11:50
@jt-dd jt-dd changed the title Ffix docker compose deployment Fix docker compose deployment Sep 13, 2024
Minosity-VR
Minosity-VR previously approved these changes Sep 13, 2024
Copy link
Contributor

@Minosity-VR Minosity-VR left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just needs a renaming because #264 was renamed too

deployments/kubehound/docker-compose.release.ingestor.yaml Outdated Show resolved Hide resolved
@jt-dd jt-dd merged commit f80397e into main Sep 13, 2024
8 checks passed
@jt-dd jt-dd deleted the jt-dd/fix-docker-compose-deployment branch September 13, 2024 12:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants