Skip to content
This repository was archived by the owner on Jul 30, 2025. It is now read-only.

Support runtime configuration in v2 #1345

@diegolovison

Description

@diegolovison

/kind feature

Description:
The current code is

RESOURCE_OP_IMAGE = ":".join(["quay.io/aipipeline/kubectl-wrapper", "latest"])
TEKTON_HOME_RESULT_PATH = "/tekton/home/tep-results/"

# The image to use in basic bash steps such as copying results in multi-step.
TEKTON_BASH_STEP_IMAGE = env.get('TEKTON_BASH_STEP_IMAGE', 'busybox')
TEKTON_COPY_RESULTS_STEP_IMAGE = env.get('TEKTON_COPY_RESULTS_STEP_IMAGE', 'library/bash')
GENERATE_COMPONENT_SPEC_ANNOTATIONS = env.get('GENERATE_COMPONENT_SPEC_ANNOTATIONS', True)

It means that it will be initialized when doing import kfp_tekton

As a developer, I wish to be able to set those variables in the runtime

os.environ["DEFAULT_STORAGE_CLASS"] = ....
os.environ["TEKTON_BASH_STEP_IMAGE"] = ...
os.environ["TEKTON_COPY_RESULTS_STEP_IMAGE"] = ...
os.environ["CONDITION_IMAGE_NAME"] = ....

If you believe that is doable having a fix in 1.5 sounds good. Otherwise, I believe that we should have this in mind for v2.

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions