Change "Check Incompatible Licenses" to "Check for Incompatible
Licenses".
This commit is made as part of a project to separate the changes in
https://github.com/ceph/ceph/pull/60448 into individual pull requests.
Co-authored-by: Anthony D'Atri <anthony.datri@gmail.com>
Signed-off-by: Zac Dover <zac.dover@proton.me>
In the absence of `name`, GH uses the `job` ID of the action to select which checks are required, and `pull_request` is rather confusing.
Signed-off-by: Ernesto Puerta <37327689+epuertat@users.noreply.github.com>
rgw: document and label mstart.sh and related scripts
Reviewed-by: Patrick Donnelly <pdonnell@ibm.com>
Reviewed-by: Shilpa Jagannath <smanjara@redhat.com>
The test job definition under qa/suites is an integral part of almost
any test. Often, the test logic is split between the task or workunit
and respective snippet(s) under qa/suites.
Other files under qa are less used, but still related to nothing but
testing, so just add the label on all of it.
Signed-off-by: Ilya Dryomov <idryomov@gmail.com>
With https://github.com/actions/labeler/issues/483 still pending, do it
manually.
A couple of entries matching both files and directories were expanded
since, unlike fnmatch(3), minimatch treats slashes specially:
- src/include/rbd* -> src/include/rbd* src/include/rbd/**
- src/rbd* -> src/rbd* src/rbd*/**
- src/test/rbd* -> src/test/rbd* src/test/rbd*/**
A bunch of entries under src/tools were combined into:
- src/tools/rbd*/**
Signed-off-by: Ilya Dryomov <idryomov@gmail.com>
Getting warning about node16 being deprecated. The workflow doesn't use node
directly, but through the external actions. Moving to node20 requires
changing setup-python version; Bhacaz/checkout-files is deprecated and
recommends actions/checkout.
Signed-off-by: Dan Mick <dmick@redhat.com>
`sync-labels` is defined as "Whether or not to remove labels
when matching files are reverted or no longer changed by the PR".
We should disable this in case someone manually adds a label,
so the label doesn't get removed.
Signed-off-by: Laura Flores <lflores@ibm.com>
Followup to commit 265f727481.
Fixes this error mesage under GitHub Insights (https://github.com/ceph/ceph/network/updates):
```
The property '#/updates/1/commit-message/prefix' was not of a maximum string length of 15
```
Signed-off-by: Laura Flores <lflores@ibm.com>
```
Error: Request failed due to following response errors:
- Could not resolve to a ProjectV2 with the number 2.
```
On further investigation, it looks like it could be a permission issue.
I followed the recommendation in their documentation to fix it: https://github.com/actions/add-to-project#creating-a-pat-and-adding-it-to-your-repository
Signed-off-by: Nizamudeen A <nia@redhat.com>
Assume ownership of qa/workunits/windows. Despite the generic name,
currently it has just rbd-wnbd tests.
Signed-off-by: Ilya Dryomov <idryomov@gmail.com>
This happens because core and rbd ownership is defined after
doc-writers: a later match takes precedence.
Signed-off-by: Ilya Dryomov <idryomov@gmail.com>
In response to https://tracker.ceph.com/issues/58173, it would have helped to have the PR label librados test files with core, so the PR would have been tagged for rados qa.
Signed-off-by: Laura Flores <lflores@redhat.com>
Currently, the stale bot closes PRs 90 days after they are marked stale since it counts "marking a PR stale" as activity.
We really want the PRs to close after a total of 90 days, or 30 days after they are marked stale. Changing `days-before-prs-close` to 30 will fix this.
Signed-off-by: Laura Flores <lflores@redhat.com>
The old (classic) Dashboard project (https://github.com/ceph/ceph/projects/6) has been migrated to a new-style one (https://github.com/orgs/ceph/projects/2/views/1).
This also requires updating the GH Action that automatically assigned `dashboard`-labeled PRs to the Dashboard project.
Signed-off-by: Ernesto Puerta <37327689+epuertat@users.noreply.github.com>
Otherwise GitHub sends an annoying e-mail right away when you file a PR that doesn't have the checklist filled out. It's easier IMO to create the PR, then check the boxes instead of putting Xes in brackets while filling out the PR comment.
Signed-off-by: David Galloway <dgallowa@redhat.com>
Zac Dover
Ernesto Puerta
Casey Bodley
Casey Bodley
Ernesto Puerta
Vallari Agrawal
Matan Breizman
Patrick Donnelly
Ilya Dryomov
Neha Ojha
Dan Mick
Laura Flores
Laura Flores
dependabot[bot]
Redouane Kachach
Anthony D'Atri
Nizamudeen A
Laura Flores
David Galloway