- 
                Notifications
    You must be signed in to change notification settings 
- Fork 0
Chore/add launches #10
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
Conversation
        
          
                README.md
              
                Outdated
          
        
      | exit | ||
|  | ||
| ## backup/dump a database | ||
| docker exec -it postgres-03939690 /bin/bash | 
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this should be crucible-postgres now?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yep
        
          
                Crucible.AppHost/AppHost.cs
              
                Outdated
          
        
      | .WithEnvironment("KC_DB", "postgres") | ||
| .WithEnvironment("KC_DB_URL_HOST", "postgres") | ||
| .WithEnvironment("KC_DB_USERNAME", "postgres") | ||
| .WithEnvironment("KC_DB_PASSWORD", postgresPassword) | 
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
instead of adding a parameter for the postgres password, we can access the generated one like this (and the host and username while we're at it)
.WithEnvironment("KC_DB_URL_HOST", postgres.Resource.PrimaryEndpoint.Property(EndpointProperty.Host))
.WithEnvironment("KC_DB_USERNAME", postgres.Resource.UserNameReference)
.WithEnvironment("KC_DB_PASSWORD", postgres.Resource.PasswordParameter)what do you think?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'll give it a try.
        
          
                README.md
              
                Outdated
          
        
      | docker exec -it postgres-03939690 /bin/bash | ||
| pg_dump -U postgres blueprint > /tmp/blueprint.dump | ||
| exit | ||
| docker cp postgres-03939690:/tmp/blueprint.dump blueprint.dump | 
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
crucible-postgres here too?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yep
named the dev container "crucible-dev"
added additional launches
fixed scopes that were not correct
replaced aschlack user with ogadmin user using the original admin user guid from the crucible beginnings
added steps to dump and restore the app databases to the README