From: Nikolai Kondrashov <spbnick@gmail.com>
To: syzkaller <syzkaller@googlegroups.com>,
Dmitry Vyukov <dvyukov@google.com>,
Vishal Bhoj <vishal.bhoj@linaro.org>,
Alice Ferrazzi <alicef@gentoo.org>,
automated-testing@lists.yoctoproject.org,
Cristian Marussi <cristian.marussi@arm.com>,
Tim Bird <Tim.Bird@sony.com>,
Johnson George <Johnson.George@microsoft.com>,
Veronika Kabatova <vkabatov@redhat.com>,
"kernelci@lists.linux.dev" <kernelci@lists.linux.dev>,
Don Zickus <dzickus@redhat.com>, Mark Brown <broonie@kernel.org>,
Philip Li <philip.li@intel.com>,
Denys Fedoryshchenko <denys.f@collabora.com>,
Michael Hofmann <mhofmann@redhat.com>,
Tales da Aparecida <tdaapare@redhat.com>,
Aditya Nagesh <adityanagesh@microsoft.com>,
Jeny Dhruvit Sheth <jeny.sadadia@collabora.com>,
Sachin Sant <sachinp@linux.ibm.com>,
"Hambardzumyan, Minas" <minas@ti.com>
Subject: Re: KCIDB: Support more checkout data
Date: Mon, 26 Aug 2024 13:39:13 +0300 [thread overview]
Message-ID: <a600ef28-67f8-40a7-87bb-886aa63f2f1a@gmail.com> (raw)
In-Reply-To: <9ea96b8e-6490-4541-81eb-8bab011c8918@gmail.com>
On 8/21/24 8:02 PM, Nikolai Kondrashov wrote:
> Hello again, everyone (potentially) involved with sending data to KCIDB,
>
> The previous schema update (v4.4, test data enhancements) is already being
> deployed, but I have prepared more backwards-compatible changes to go out with
> v4.5.
>
> This time I rolled everything into a single commit, but I'll split it up, if
> we have disagreements, or need to work more on the changes.
>
> The PR is at https://github.com/kernelci/kcidb-io/pull/87
>
> But I'll repeat the commit message here:
>
> Add support for three more checkout fields: `git_commit_tags`,
> `git_commit_message`, and `git_repository_branch_tip`.
>
> The `git_commit_tags` is an array of strings representing annotated tags
> pointing directly at the commit being checked out, as seen in the source
> repository. I.e. the output of `git tag --points-at <commit>`. Set to an empty
> array, if the commit has no tags.
>
> The `git_commit_message` is intended to hold the complete message of the
> commit being checked out, *both* subject and body. I.e. the output of `git
> show -s --format=%B`. We're putting the subject and the body together, as it's
> quite easy to extract the subject in SQL, while full-text search is easier and
> more efficient to do over a single column.
>
> Finally, the `git_repository_branch_tip` is a boolean flag, which should be
> set to `true`, when the commit being checked out is at the tip of the branch
> at the moment of the checkout (as specified in `start_time`). Essentially, if
> you're always testing only the tip of the branch, you can set this to `true`
> unconditionally. This flag would let us extract the checkouts which
> represented the branch state over time, and produce a rough history of branch
> changes, which we can then use for (regression) analysis and graphs, in lieu
> of actual commit graph walking.
>
> The (abbreviated) schema for all three fields is below and is very simple:
>
> "git_commit_tags": {
> "type": "array",
> "items": {"type": "string"},
> },
> "git_commit_message": {"type": "string"},
> "git_repository_branch_tip": {"type": "boolean"}
>
> The PR has the complete schema, with inline docs.
I'm going to merge this and start work on the support on Wednesday, Aug 28, if
there are no objections by that time.
Nick
next prev parent reply other threads:[~2024-08-26 10:39 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-08-21 17:02 KCIDB: Support more checkout data Nikolai Kondrashov
2024-08-26 10:39 ` Nikolai Kondrashov [this message]
2024-09-16 11:43 ` Nikolai Kondrashov
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=a600ef28-67f8-40a7-87bb-886aa63f2f1a@gmail.com \
--to=spbnick@gmail.com \
--cc=Johnson.George@microsoft.com \
--cc=Tim.Bird@sony.com \
--cc=adityanagesh@microsoft.com \
--cc=alicef@gentoo.org \
--cc=automated-testing@lists.yoctoproject.org \
--cc=broonie@kernel.org \
--cc=cristian.marussi@arm.com \
--cc=denys.f@collabora.com \
--cc=dvyukov@google.com \
--cc=dzickus@redhat.com \
--cc=jeny.sadadia@collabora.com \
--cc=kernelci@lists.linux.dev \
--cc=mhofmann@redhat.com \
--cc=minas@ti.com \
--cc=philip.li@intel.com \
--cc=sachinp@linux.ibm.com \
--cc=syzkaller@googlegroups.com \
--cc=tdaapare@redhat.com \
--cc=vishal.bhoj@linaro.org \
--cc=vkabatov@redhat.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox