Conversation
|
I am not sure why the actions launched the |
427cb2b to
5eae06d
Compare
It seems it was just an automatic check of the correctness of the |
4437fe8 to
94a562f
Compare
700c34b to
8b0f413
Compare
|
I will have to add a few diagrams here, hence, switching to draft. |
ed03c87 to
55245e6
Compare
Signed-off-by: Danil Klimuk <daniil.klimuk@3mdeb.com>
woodpecker-cli Signed-off-by: Danil Klimuk <daniil.klimuk@3mdeb.com>
Signed-off-by: Danil Klimuk <daniil.klimuk@3mdeb.com>
This is because it is being used in other TrenchBoot repos as well but on other branches because of: TrenchBoot/grub#32 TrenchBoot/qubes-antievilmaid#15 TrenchBoot/xen#26 IMHO there is no reason to mention every use of this workflow. Signed-off-by: Danil Klimuk <daniil.klimuk@3mdeb.com>
55245e6 to
0de7bf6
Compare
Moving to "Ready for review" now, I will consider adding the diagrams to TrenchBoot docs instead, as the diagrams will describe an entire CI/CD infrastructure for automatic rebasing in TrenchBoot organization. This seems to be a good place: https://trenchboot.org/dev-docs/ . |
| with: | ||
| repository: TrenchBoot/.github | ||
| path: shared | ||
| ref: master |
There was a problem hiding this comment.
Wouldn't you want the same ref as this workflow was called with?
There was a problem hiding this comment.
No I do not see a reason, unless there is a custom change to the rebase.sh that should be tested. But I have not found the way to redirect to the reusable workflow the ref it was called from. Because the reusable workflow inherits all the environment variables from the parent workflow (such as, for example github.sha, etc.). So for me the only way here is to use inputs to tell the reusable workflow about it, but I do not want to create yet another input just for that.
There was a problem hiding this comment.
You are right, I got a little confused with how this would work with reusable workflow (and noticed there are GitHub issues related to exactly this)
There was a problem hiding this comment.
No I do not see a reason, unless there is a custom change to the
rebase.sh
I can't test it as rebase.sh doesn't exist on master currently :(
There was a problem hiding this comment.
I have unresolved this thread, as it seems I have found the solution: https://docs.github.com/en/actions/reference/workflows-and-actions/contexts#example-usage-of-job-context-workflow-identity .
There was a problem hiding this comment.
|
I will need to do a squash at the end. |
Signed-off-by: Danil Klimuk <daniil.klimuk@3mdeb.com>
69ac4e4 to
ed96075
Compare
Signed-off-by: Danil Klimuk <daniil.klimuk@3mdeb.com>
|
I have found another corner case: 9501536 . |
Signed-off-by: Danil Klimuk <daniil.klimuk@3mdeb.com>
| NAME: ${{ inputs.commit-user-name }} | ||
| EMAIL: ${{ inputs.commit-user-email }} | ||
| MESSAGE: ${{ inputs.cicd-trigger-resume }} | ||
| run: | |
There was a problem hiding this comment.
I think we should deal with return code 5 (nothing to rebase). I don't think failing job (red x) would look good. I think in that case rest of the jobs should be skipped
There was a problem hiding this comment.
IMO differentiating between a rebase success and nothing to rebase could be useful, so instead of modifying the script, I have modified the reusable workflow: b70dcd6 . But now I need to check the rebase-exit-code in the calling workflows.
There was a problem hiding this comment.
I have modified all workflows that use this reusable workflow so far:
- https://github.com/TrenchBoot/grub/compare/54dee6b34d817d1d847757985cf00f15c6488a8a..f69ced86bdff240364570e59ef8ea20561d6702d
- https://github.com/TrenchBoot/qubes-antievilmaid/compare/703c199cecfdff5094cb42ed289e01735456481f..fcf825c6925d3ce39d75b75054ecefeea763fcab
- https://github.com/TrenchBoot/xen/compare/ae91a9bac0ae3a17155f044417ee34a044103280..dc90373568fb0051d8460943db8e0207641abfec
Signed-off-by: Danil Klimuk <daniil.klimuk@3mdeb.com>
8a88744 to
2ba4cf6
Compare
Signed-off-by: Danil Klimuk <daniil.klimuk@3mdeb.com>
2ba4cf6 to
e51d845
Compare
code 5 should be treated as a success Signed-off-by: Danil Klimuk <daniil.klimuk@3mdeb.com>
No description provided.