Skip to content
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 plumbing dependencies to fix CI #1374

Merged
merged 1 commit into from
May 21, 2021

Conversation

vinamra28
Copy link
Member

Changes

Signed-off-by: vinamra28 vinjain@redhat.com

Submitter Checklist

These are the criteria that every PR should meet, please check them off as you
review them:

  • Includes tests (if functionality changed/added)
  • Run the code checkers with make check
  • Regenerate the manpages, docs and go formatting with make generated
  • Commit messages follow commit message best practices

See the contribution guide
for more details.

Release Notes

NONE

@tekton-robot tekton-robot added the release-note-none Denotes a PR that doesnt merit a release note. label May 19, 2021
@tekton-robot tekton-robot added the size/M Denotes a PR that changes 30-99 lines, ignoring generated files. label May 19, 2021
@vinamra28
Copy link
Member Author

/retest

Copy link
Member

@vdemeester vdemeester left a comment

Choose a reason for hiding this comment

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

/lgtm

@tekton-robot tekton-robot added the lgtm Indicates that a PR is ready to be merged. label May 19, 2021
@tekton-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: 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 /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@tekton-robot tekton-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label May 19, 2021
@vinamra28
Copy link
Member Author

/retest

@vinamra28 vinamra28 force-pushed the bump-plumbing-scripts branch from 0b09155 to 7da7e49 Compare May 20, 2021 01:55
@tekton-robot tekton-robot removed the lgtm Indicates that a PR is ready to be merged. label May 20, 2021
@vinamra28 vinamra28 force-pushed the bump-plumbing-scripts branch from 7da7e49 to 0462fce Compare May 20, 2021 05:45
@vinamra28
Copy link
Member Author

/retest

1 similar comment
@vdemeester
Copy link
Member

/retest

@vinamra28 vinamra28 force-pushed the bump-plumbing-scripts branch from 0462fce to ca8c3ca Compare May 20, 2021 07:09
@vdemeester
Copy link
Member

/lgtm

@tekton-robot tekton-robot added the lgtm Indicates that a PR is ready to be merged. label May 20, 2021
@vinamra28 vinamra28 mentioned this pull request May 20, 2021
4 tasks
@vinamra28
Copy link
Member Author

/retest

@vinamra28 vinamra28 force-pushed the bump-plumbing-scripts branch from ca8c3ca to 07b6716 Compare May 20, 2021 11:37
@tekton-robot tekton-robot removed the lgtm Indicates that a PR is ready to be merged. label May 20, 2021
@vinamra28
Copy link
Member Author

/hold

@tekton-robot tekton-robot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label May 20, 2021
@vinamra28
Copy link
Member Author

/retest

@vinamra28
Copy link
Member Author

/test pull-tekton-cli-integration-tests

1 similar comment
@vinamra28
Copy link
Member Author

/test pull-tekton-cli-integration-tests

@vinamra28 vinamra28 force-pushed the bump-plumbing-scripts branch from 07b6716 to 659fcf7 Compare May 20, 2021 16:04
@vinamra28
Copy link
Member Author

/retest

@vinamra28
Copy link
Member Author

/retest

@vinamra28 vinamra28 force-pushed the bump-plumbing-scripts branch 8 times, most recently from 0bb995c to 2000f6c Compare May 21, 2021 06:39
@vinamra28
Copy link
Member Author

/retest

Signed-off-by: vinamra28 <vinjain@redhat.com>
@vinamra28 vinamra28 force-pushed the bump-plumbing-scripts branch from 2000f6c to d68e08b Compare May 21, 2021 06:55
Copy link
Contributor

@savitaashture savitaashture left a comment

Choose a reason for hiding this comment

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

/lgtm

@tekton-robot tekton-robot added the lgtm Indicates that a PR is ready to be merged. label May 21, 2021
@vinamra28
Copy link
Member Author

/hold cancel

@tekton-robot tekton-robot removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label May 21, 2021
@tekton-robot tekton-robot merged commit 951b5be into tektoncd:main May 21, 2021
@vinamra28 vinamra28 deleted the bump-plumbing-scripts branch May 21, 2021 07:19
piyush-garg added a commit that referenced this pull request Jun 8, 2021
#1361 | [Pradeep Kumar] update cli version to v0.18.0 in readme | 2021/04/27-17:00
#1367 | [PuneetPunamiya] Updates TriggerTemplate Tests from Test Builders to Structs | 2021/05/10-11:28
#1367 | [PuneetPunamiya] Updates TriggerBinding Tests from Test Builders to Structs | 2021/05/10-11:28
#1367 | [PuneetPunamiya] Updates ClusterTriggerBinding Tests from Test Builders to Structs | 2021/05/10-11:28
#1367 | [PuneetPunamiya] Updates Eventlistener Tests from Test Builders to Structs | 2021/05/10-11:28
#1364 | [SM43] Fixes tkn task start to ask for params interactively if not passed | 2021/05/12-06:53
#1366 | [Chmouel Boudjnah] Lookup plugins in current path | 2021/05/12-12:59
#1374 | [vinamra28] Bump plumbing dependencies to fix CI | 2021/05/21-08:18
#1375 | [SM43] Fixes e2e test for ClusterTask | 2021/05/24-05:41
#1369 | [SM43] Bump Hub dependency with latest updates | 2021/05/25-16:30
null | [Piyush Garg] Bump tektoncd/pipeline to v0.24.1 | 2021/06/01-14:47
null | [Anshul Verma] Fixes Issue #1067     Added --component option to just output the version of the components like client|pipeline|triggers | 2021/06/01-15:28
null | [Anshul Verma] Fixes 1000 Interative pipeline/task start fail when `-f` option is used and a `param` in the definition does not have the `type` defined. This fix gives a validation error if any `param` does not have a type defined. | 2021/06/02-10:34
null | [SM43] Fixes pipeline start to ask for params interactively if not passed | 2021/06/02-12:14
null | [SM43] Fixes clusterTask start to ask for params interactively if not passed | 2021/06/02-13:02
null | [vinamra28] Add support for optional workspaces in tkn start | 2021/06/02-13:29
null | [Piyush Garg] Bump to tektoncd/triggers v0.14.1 | 2021/06/02-14:59
null | [Piyush Garg] Fix rpm spec file | 2021/06/02-15:38
null | [vinamra28] Common Description for workspace flag in start command | 2021/06/02-19:37
null | [Chmouel Boudjnah] Add a flag to not print the taskname/step prefix when showing logs | 2021/06/03-08:59
null | [Chmouel Boudjnah] Replace maze.io/x/duration with its redirect | 2021/06/08-09:37

Signed-off-by: Piyush Garg <pgarg@redhat.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. lgtm Indicates that a PR is ready to be merged. release-note-none Denotes a PR that doesnt merit a release note. size/M Denotes a PR that changes 30-99 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants