Skip to content

Conversation

@ilan-pinto
Copy link
Member

This is the 1st commit message:

support image rendering for join command

added image rending to the join command

Signed-off-by: Ilan Pinto ilpinto@redhat.com

 This is the 1st commit message:

support image rendring for join command

added image rending to the join command

Signed-off-by: Ilan Pinto <ilpinto@redhat.com>
@@ -0,0 +1,20 @@
{
// Use IntelliSense to learn about possible attributes.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

would you remove this .vscode?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done

Signed-off-by: Ilan Pinto <ilpinto@redhat.com>
spec:
registrationImagePullSpec: quay.io/open-cluster-management/registration
workImagePullSpec: quay.io/open-cluster-management/work
registrationImagePullSpec: {{ .ImageRegistry.Registry }}/registration
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

sorry to ask again...but the version should be added here also.

Signed-off-by: Ilan Pinto <ilpinto@redhat.com>
@qiujian16
Copy link
Member

/approve
/lgtm

@openshift-ci
Copy link

openshift-ci bot commented Dec 9, 2021

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: ilan-pinto, qiujian16

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 /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-ci openshift-ci bot added the approved label Dec 9, 2021
@openshift-merge-robot openshift-merge-robot merged commit 1b0e9b8 into open-cluster-management-io:main Dec 9, 2021
@ilan-pinto ilan-pinto deleted the join-render-image branch December 10, 2021 06:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants