-
Notifications
You must be signed in to change notification settings - Fork 59
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 new SRCPVCNAME and SRCPVCNAMESPACE params #168
Conversation
ea51b6a
to
dd81581
Compare
|
||
- name: SRCPVCNAME | ||
description: Name of the PVC to clone | ||
value: '{{ oslabels | last }}' |
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.
What values would occur here? Can you give a few examples?
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.
e.g.
rhel8.2
rhel7.9
centos8
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.
Are all of these valid PVC names?
I suspect that anyething with a dot is not.
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.
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.
Names with dot are valid k8s names
dd81581
to
b159516
Compare
templates/_linux.yaml
Outdated
value: '{{ oslabels | last }}' | ||
- name: SRC_PVC_NAMESPACE | ||
description: Namespace of the source PVC | ||
value: openshift-kubevirt-base-images |
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.
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.
Fine for me
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.
fixed
These new params are added to unblock UI. They are not used in common templates yet. In the future they will be used for datavolume template. Signed-off-by: Karel Simon <ksimon@redhat.com>
b159516
to
2750cfb
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
/retest |
/cc @shwetaap |
|
This will be updated in d/s automation pipeline |
ok got it. Thanks! |
/lgtm |
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: omeryahud The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
add new SRC_PVC_NAME and SRC_PVC_NAMESPACE params to all templates.
These new params are added to unblock UI. They are not used in common templates
yet. In the future they will be used for datavolume template.
Signed-off-by: Karel Simon ksimon@redhat.com