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

generate-run-command handles --pass-config in a wrong way #2071

Closed
YevheniiSemendiak opened this issue Apr 1, 2021 · 0 comments · Fixed by #2074
Closed

generate-run-command handles --pass-config in a wrong way #2071

YevheniiSemendiak opened this issue Apr 1, 2021 · 0 comments · Fixed by #2074
Labels
bug Something isn't working CLIents UX

Comments

@YevheniiSemendiak
Copy link
Contributor

Description:

If you run a job with --pass-config enabled and afterward, use generage-run-command, the resulted command will contain:

  • the--pass-config flag, which under the hood uploads config to storage, and mounts it to the job with volume and ENV variable.
  • a pass-config-related volume and ENV variable from previous run

As a result, the setup will be conflicting and you will not be able to re-run the job.

Example:

  1. `neuro run --pass-config --name test-pass-config ubuntu bash -c "exit 0"
  2. eval $(neuro job generate-run-command test-pass-config) - fails.
    Also, the error reported by CLI is not clear:

image

Expected behavior:

generate-run-command does not mount previous --pass-config volume and ENV.

Additional context:

Neuro Platform Client 21.3.3

@YevheniiSemendiak YevheniiSemendiak added bug Something isn't working CLIents UX labels Apr 1, 2021
@YevheniiSemendiak YevheniiSemendiak changed the title neuro job generate-run-command handles --pass-config in a wrong way generate-run-command handles --pass-config in a wrong way Apr 1, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working CLIents UX
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant