From: michael.opdenacker@bootlin.com
To: bitbake-devel@lists.openembedded.org
Cc: Michael Opdenacker <michael.opdenacker@bootlin.com>
Subject: [PATCH v2 00/12] prserv: add support for an "upstream" server
Date: Fri, 5 Apr 2024 18:41:13 +0200 [thread overview]
Message-ID: <20240405164125.1652579-1-michael.opdenacker@bootlin.com> (raw)
From: Michael Opdenacker <michael.opdenacker@bootlin.com>
This makes it possible to customize an "upstream" distribution
by modifying local packages. If the "upstream" package bears
revision "x", the local one will have revision "x.y", this
having priority over the upstream one.
Multiple levels of upstream servers are supported, so "x.y.z" revisions
are possible too.
Take advantage of this work to clean-up and update the prserv code too.
Manual "sanity" tests have been run so far but automated tests
(based on Python unittest, as for the hash server tests)
are being prepared for the next iteration.
Code reviews are welcome, especially for more Pythonic
ways of implementing some details.
---
Changes in V2:
- Add this new commit:
prserv: remove unused "hist" mode in the database backend
- Squash commit "prserv: fix read_only test" into
commit "prserv: simplify the PRServerClient() interface"
(Reported by Richard Purdie)
- Fix the code to support increasing "x.y.z" values, thus
supporting several levels of upstream servers.
- db.py: remove duplicate definition of find_max_value() function in db.py
- prserv.py: remove tabs before comments (Python didn't complain)
- db.py: now stores the revision ("value") as TEXT.
This way we can store "1.0" without having it transformed to "1"
when the default type was INTEGER.
- This allows to fix a regression when the first packages were created
with 'r0.1' instead of 'r0.0' initially.
- find_max_value: now returns None instead of '0' when no value is found
Before we couldn't tell the difference between a '0'
max value and the absence of such a value.
Michael Opdenacker (12):
prserv: simplify the PRServerClient() interface
prserv: use double quotes by default
bitbake-prserv: replace deprecated optparse by argparse
prserv: use self.logger instead of logger directly
asyncrpc: include parse_address from hashserv
prserv: capitalization and spacing improvements
prserv: remove unused "hist" mode in the database backend
prserv: add extra requests
prserv: remove redundant exception handler
prserv: correct error message
prserv: remove unnecessary code
prserv: add "upstream" server support
bin/bitbake-prserv | 99 ++++++++++-----
lib/bb/asyncrpc/client.py | 23 ++++
lib/hashserv/__init__.py | 27 +---
lib/prserv/__init__.py | 21 +++-
lib/prserv/client.py | 42 +++++--
lib/prserv/db.py | 251 +++++++++++++++++---------------------
lib/prserv/serv.py | 219 ++++++++++++++++++++++++---------
7 files changed, 415 insertions(+), 267 deletions(-)
--
2.34.1
next reply other threads:[~2024-04-05 16:41 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-04-05 16:41 michael.opdenacker [this message]
2024-04-05 16:41 ` [PATCH v2 01/12] prserv: simplify the PRServerClient() interface michael.opdenacker
2024-04-05 16:41 ` [PATCH v2 02/12] prserv: use double quotes by default michael.opdenacker
2024-04-05 16:41 ` [PATCH v2 03/12] bitbake-prserv: replace deprecated optparse by argparse michael.opdenacker
2024-04-05 16:41 ` [PATCH v2 04/12] prserv: use self.logger instead of logger directly michael.opdenacker
2024-04-05 16:41 ` [PATCH v2 05/12] asyncrpc: include parse_address from hashserv michael.opdenacker
2024-04-05 16:41 ` [PATCH v2 06/12] prserv: capitalization and spacing improvements michael.opdenacker
2024-04-05 16:41 ` [PATCH v2 07/12] prserv: remove unused "hist" mode in the database backend michael.opdenacker
2024-04-05 16:41 ` [PATCH v2 08/12] prserv: add extra requests michael.opdenacker
2024-04-05 16:41 ` [PATCH v2 09/12] prserv: remove redundant exception handler michael.opdenacker
2024-04-05 16:41 ` [PATCH v2 10/12] prserv: correct error message michael.opdenacker
2024-04-05 16:41 ` [PATCH v2 11/12] prserv: remove unnecessary code michael.opdenacker
2024-04-05 16:41 ` [PATCH v2 12/12] prserv: add "upstream" server support michael.opdenacker
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=20240405164125.1652579-1-michael.opdenacker@bootlin.com \
--to=michael.opdenacker@bootlin.com \
--cc=bitbake-devel@lists.openembedded.org \
/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;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).