Skip to content

[github-workflow] Gate PR ELD toolset upload on zephyr-check label#1431

Open
quic-areg wants to merge 1 commit into
qualcomm:mainfrom
quic-areg:zephyr-check-label-gate
Open

[github-workflow] Gate PR ELD toolset upload on zephyr-check label#1431
quic-areg wants to merge 1 commit into
qualcomm:mainfrom
quic-areg:zephyr-check-label-gate

Conversation

@quic-areg

Copy link
Copy Markdown
Contributor

97ea3dd disabled the upload because it ran on every CI build and the storage utilization ended up being too high (~25 MB * ~30 runs/day at 1 day retention).

Re-enable, gated on a zephyr-check PR label.

Requires the zephyr-check label to exist on qualcomm/eld.

97ea3dd disabled the upload because it ran on every CI build and the
storage utilization ended up being too high (~25 MB * ~30 runs/day
at 1 day retention).

Re-enable, gated on a `zephyr-check` PR label.

Requires the `zephyr-check` labelto exist on qualcomm/eld.

Signed-off-by: quic-areg <aregmi@qti.qualcomm.com>
Comment thread .github/workflows/ci.yml
- name: Upload PR ELD toolset
if: |
steps.file-check.outputs.skip_build == 'false' &&
contains(github.event.pull_request.labels.*.name, 'zephyr-check')

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

nit : this can be converted to a step that evaluates a condition and that variable can be used across the board, that will reduce repeating this across the workflow.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants