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

Persist docker deployment infos to a file in the current directory #4109

Merged
merged 1 commit into from
Jun 6, 2022

Conversation

fmhoeger
Copy link
Collaborator

@fmhoeger fmhoeger commented Jun 3, 2022

When deploying via docker important information is written to the console (STDOUT) at the end of the deployment process. This includes the URL of the deployed experiment, the credentials to log into the dashboard as well as info for how to access the logs.

Unfortunately, all this data is lost when closing the terminal window, e.g. unintentionally, unless it has been manually copied into a file before.

We want the deployment information as described above to be automatically written to a file in the current directory.

See #4096

@codecov
Copy link

codecov bot commented Jun 3, 2022

Codecov Report

Merging #4109 (b466967) into master (ecbd5ee) will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff           @@
##           master   #4109   +/-   ##
======================================
  Coverage    76.8%   76.8%           
======================================
  Files          47      47           
  Lines        8588    8588           
  Branches     1280    1280           
======================================
  Hits         6587    6587           
  Misses       1674    1674           
  Partials      327     327           

@fmhoeger fmhoeger requested a review from pmcharrison June 3, 2022 02:58
@fmhoeger fmhoeger merged commit 228acac into master Jun 6, 2022
@fmhoeger fmhoeger deleted the 4096_persist-docker-deployment-info branch June 6, 2022 22:38
@fmhoeger fmhoeger restored the 4096_persist-docker-deployment-info branch June 8, 2022 05:52
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.

None yet

2 participants