Skip to content

Prefix container names with sublime_ - #70

Merged
hugh-sublime merged 3 commits into
mainfrom
hugh.container-name-prefix
Jan 25, 2023
Merged

hugh-sublime merged 3 commits into
mainfrom
hugh.container-name-prefix

Conversation

@hugh-sublime

@hugh-sublime hugh-sublime commented Jan 24, 2023 •

Copy link
Copy Markdown
Member

There may be collisions with existing Docker containers (e.g. postgres) so this prefixes all Docker containers with sublime_.

Tested locally with clone_platform=false ./install-and-launch.sh

@hugh-sublime
hugh-sublime requested review from a team and jkamdjou January 24, 2023 22:10
Comment thread docker-compose.yml Outdated
image: sublimesec/bora-lite:0.39
restart: unless-stopped
container_name: bora-lite
container_name: sublime_bora-lite

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

while we're at it... any objections to using underscores for everything for consistency?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Just went ahead and added sublime_ to everything basically for consistency including the service names.

Comment thread docker-compose.yml Outdated
- AWS_REGION=us-east-1
- DISABLE_DD=true
container_name: screenshot-service
container_name: sublime_screenshot-service

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

ditto here

@cameron-dunn-sublime cameron-dunn-sublime left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Should we rename sublimes3 to sublime_s3 as well?

Comment thread docker-compose.yml Outdated
Comment thread docker-compose.yml
minio server --address 0.0.0.0:8110 --console-address 0.0.0.0:8111 /data;
"
create-buckets:
sublime_create_buckets:

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

@cameron-dunn-sublime this was the only container with something that kind of resembled an error:

2023-01-24 14:38:45 mc: <ERROR> Unable to initialize new alias from the provided credentials. Invalid Request (invalid hostname).
2023-01-24 14:38:45 Bucket created successfully `myminio/email-screenshots`.
2023-01-24 14:38:45 [2023-01-24 22:38:45 UTC] 4.0KiB email-screenshots/

Is this expected or do I need to update something here?

Comment thread docker-compose.yml
- create-buckets
- sublime_create_buckets
# Keep this name as sublimes3 because underscores don't play nice with certain endpoint validation
sublimes3:

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

@cameron-dunn-sublime helped me verify that using an underscore here causes an invalid host name. We verified everything else!

distribution: 600
scanner: 150
coordinator:
addr: 'strelka_coordinator_1:6379'

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

does this underscore have issues too? should we use dashes?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I checked the Strelka backend container logs and it looks like it finds the coordinator:

2023-01-24 15:35:52 2023-01-24 23:35:52 - [INFO] root [strelka-backend.main]: using backend configuration /etc/strelka/backend.yaml
2023-01-24 15:35:52 2023-01-24 23:35:52 - [DEBUG] root [strelka-backend.main]: verified coordinator is up

@hugh-sublime
hugh-sublime merged commit eec21d2 into main Jan 25, 2023
@hugh-sublime
hugh-sublime deleted the hugh.container-name-prefix branch January 25, 2023 16:16
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.

4 participants