-
Notifications
You must be signed in to change notification settings - Fork 187
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
Add tag name argument to graminize #662
Conversation
ef736cd
to
fb75849
Compare
da97606
to
08cc9a9
Compare
df5cd13
to
9e1f899
Compare
openfl/interface/workspace.py
Outdated
@@ -443,6 +443,11 @@ def dockerize_(context, base_image, save): | |||
'Must be a power-of-2.\n' | |||
'Default is 16G.' | |||
) | |||
@option('-t', '--tag', required=False, | |||
type=str, multiple=False, default=None, |
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.
type=str, multiple=False, default=None, | |
type=str, multiple=False, default='', |
openfl/interface/workspace.py
Outdated
@@ -501,6 +506,10 @@ def open_pipe(command: str): | |||
|
|||
workspace_path = Path.cwd() | |||
workspace_name = workspace_path.name | |||
|
|||
if tag is None: |
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.
if tag is None: | |
if not tag: |
Allow seting the image tag in "fx workspace graminize" using the "-t" argument. When set, it will be used instead of the workspace name Docker image tag and archive file name Signed-off-by: Lavi, Nir <nir.lavi@intel.com>
34b3d19
to
9808dc7
Compare
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.
LGTM!
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.
LGTM!
Set the graminized image name in "fx workspace graminize" using the "-t" argument. The tag name is formatted with the "worspace_name" input, such that "{workspace_name}" format specifier in the tag name will be replace by the workspace name
If not specified, the workspace name will be used as tag (current behavior)
Signed-off-by: Lavi, Nir nir.lavi@intel.com