-
Notifications
You must be signed in to change notification settings - Fork 238
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
Bump pipeline and triggers deps #1225
Conversation
/hold |
c1f9cb2
to
c403ace
Compare
/hold cancel |
/test pull-tekton-cli-integration-tests-0_10 |
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.
/meow
In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
c403ace
to
761edef
Compare
pkg/cmd/eventlistener/describe.go
Outdated
TEMPLATE REF APIVERSION | ||
{{ decorate "bullet" $v.Template.Ref }} {{ $v.Template.APIVersion }} | ||
{{- else }} | ||
TEMPLATE NAME APIVERSION |
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.
Looks good to me. Only thing left might be to match that spacing for TEMPLATE NAME
and APIVERSION
. It's oddly long, but the spacing should be consistent.
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.
I am not sure about this, but both TEMPLATE NAME APIVERSION
and TEMPLATE REF APIVERSION
have one tab space.
761edef
to
ee59b03
Compare
/retest |
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.
Just some nits then I think this is good.
NAME VALUE | ||
{{- range $b := $v.Bindings }} | ||
{{- if ne $b.Name "" }} | ||
{{ decorate "bullet" $b.Name }} {{ $b.Value }} |
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.
Wasn't sure what to make of this, but the example [here](kubectl apply -f https://raw.githubusercontent.com/tektoncd/triggers/master/examples/eventlisteners/eventlistener-embedded-bindings.yaml) comes back with a <nil>
for value:
Name: listener-embed-binding
Namespace: default
Service Account: tekton-triggers-example-sa
URL: http://el-listener-embed-binding.default.svc.cluster.local:8080
EventListnerServiceName: el-listener-embed-binding
EventListenerTriggers
NAME
∙ foo-trig
BINDINGS
REF KIND APIVERSION
∙ pipeline-binding
NAME VALUE
∙ message-binding <nil>
SPEC
∙ Params
message Hello from the Triggers EventListener(listener-embed-binding)!
TEMPLATE REF APIVERSION
∙ pipeline-template
I suppose we can leave it for now, but might be nice to clean up in future.
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.
definitely, we will need cleanup, as the deprecated part becomes unsupported.
This bumps triggers to 0.9.1 For pipeline its already taken care in [tektoncd#1198](tektoncd#1198) Note: takes care of deprecations will need to clean after deprecations are not support Signed-off-by: Pradeep Kumar <pradkuma@redhat.com>
ee59b03
to
269b9bb
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
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: danielhelfand, vdemeester 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 |
/retest |
This bumps triggers to 0.9.1
For pipeline its already taken care in
#1198
Signed-off-by: Pradeep Kumar pradkuma@redhat.com
Changes
Submitter Checklist
These are the criteria that every PR should meet, please check them off as you
review them:
make check
make generated
See the contribution guide
for more details.
Release Notes