-
Notifications
You must be signed in to change notification settings - Fork 30
Conversation
bentoctl/deployment_config.py
Outdated
"type": { | ||
"required": True, | ||
"default": "terraform", | ||
"help_message": "The type of deployment project to use", | ||
} |
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.
change it to template_type
bentoctl/deployment_config.py
Outdated
v = cerberus.Validator() | ||
validated_spec = v.validated(copied_operator_spec, schema=operator_schema) | ||
if validated_spec is None: | ||
raise InvalidDeploymentConfig(config_errors=v.errors) | ||
self.operator_spec = validated_spec | ||
|
||
def generate(self): | ||
self.operator.generate( |
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.
check the template type
bentoctl/cli/__init__.py
Outdated
build_docker_image( | ||
image_tag=image_tag, | ||
context_path=dockercontext_path, | ||
dockerfile=dockerfile_path, | ||
additional_build_args=build_args, | ||
) |
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 determined by the push flag, if user decided not to push, we can also create a docker image file in the current directory (that could be a flag too)
bentoctl/cli/__init__.py
Outdated
@click.option("--registry-password") | ||
@click.option("--registry-username") | ||
@click.option("--debug", is_flag=True, default=False) | ||
def build( | ||
bento_tag, | ||
deployment_config_file, | ||
push, | ||
overwrite_deployable, | ||
registry_username, | ||
registry_password, | ||
debug, |
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.
we already have debug functionality setup for all commands. Its sets an env-var to signify it and changes logging mode to debug
bentoctl/deployment_config.py
Outdated
bento_name=self.bento.tag.name, | ||
spec=self.operator_spec, | ||
) | ||
self.operator_spec["repository_name"] = self.bento.tag.name |
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.
@jjmachan make sure we need this, check it with lambda
No description provided.