pipeline
pipeline copied to clipboard
[FIX] Remove the apt warning
Changes
The apt manpage recommends against using it in scripts and issues a warning. This patch fixes the warning.
It also adds an option to debug ci worlflow via workflow_dispatch, and removes yamllint exception by properly formatting the on clause.
Submitter Checklist
As the author of this PR, please check off the items in this checklist:
- [x] Has Docs if any changes are user facing, including updates to minimum requirements e.g. Kubernetes version bumps
- [x] Has Tests included if any functionality added or changed
- [x] pre-commit Passed
- [x] Follows the commit message standard
- [x] Meets the Tekton contributor standards (including functionality, content, code)
- [x] Has a kind label. You can add one by adding a comment on this PR that contains
/kind <type>. Valid types are bug, cleanup, design, documentation, feature, flake, misc, question, tep - [x] Release notes block below has been updated with any user facing changes (API changes, bug fixes, changes requiring upgrade notices or deprecation warnings). See some examples of good release notes.
- [x] Release notes contains the string "action required" if the change requires additional action from users switching to the new release
Release Notes
NONE
The committers listed above are authorized under a signed CLA.
- :white_check_mark: login: leshikus / name: Alexei Fedotov (852523563bbdfcd6834a6905a23fe92784a5be86, c284adb9d40afb038d384b2ed67052829f6a6983, 084554f7b9394b7365e1041d2da1c5eb88eda874)
- :white_check_mark: login: waveywaves / name: Vibhav Bobade (59647df25ee70d2c6d63605e9470275933527cab)
/kind cleanup
@pritidesai how can I make progress with this PR?
@vdemeester @afrittoli , can we progress with this patch? I was thinking about spending some time working on this project and resolving more serious issues in future.
[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
- ~~OWNERS~~ [vdemeester]
Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment
I wonder how can this PR get this LGTM label
@leshikus can you take care of my last comment, and I'll LGTM πΌπΌ π
@leshikus I have updated the PR with the required change on your behalf hope that is ok, this is so that we can fast-forward this PR :)
cc @vdemeester
@waveywaves thanks! sorry for missing the previous request
/lgtm ing based on Vincent's last comment π