Earlier patches changed a couple of function signatures and did not
update all callers. These issues were caught with manual testing.
Change-Id: Ib3869c19b945fbcb9ffe46e1abec5cf5a0a2897f
Signed-off-by: Doug Hellmann <doug@doughellmann.com>
Now that get_deliverables() returns Deliverable objects instead of
tuples we don't have to turn the tuples into objects.
Change-Id: I83c6da75f6186cf7001baa8e162ad6d5714cc3e7
Signed-off-by: Doug Hellmann <doug@doughellmann.com>
Now that get_deliverables() returns Deliverable objects instead of
tuples we don't have to turn the tuples into objects.
Change-Id: I26066c0dc891b0a226097154b0e2e33293d09e44
Signed-off-by: Doug Hellmann <doug@doughellmann.com>
Now that get_deliverables() returns Deliverable objects instead of
tuples we don't have to turn the tuples into objects.
Change-Id: Iaf13cde694692bfcee52349ec14dc7e15b6afd78
Signed-off-by: Doug Hellmann <doug@doughellmann.com>
Now that get_deliverables() returns Deliverable objects instead of
tuples we don't have to turn the tuples into objects.
Change-Id: I42c826a82099dd9e3b26253385dd565b2948763f
Signed-off-by: Doug Hellmann <doug@doughellmann.com>
get_deliverables() now returns Deliverable objects instead of tuples,
so we don't need to convert the tuples to objects.
Change-Id: I28d49a64c443b22f102536a093b59a917db65457
Signed-off-by: Doug Hellmann <doug@doughellmann.com>
Rather than applying the rules ourselves, use the packaging project
utility function to prepare the canonical name.
Change-Id: I6310585ff14c376e954b0edc9def67cd65a5d550
Signed-off-by: Doug Hellmann <doug@doughellmann.com>
An earlier change in the series removed the static data being used for
configuring the command line interface of list-deliverables. Replace
that with something that reads the schema file instead so we only need
to have the settings in one place.
Change-Id: Ib40e3bdff2f127cf5041dfffb03d53c0d1a6f3a0
Signed-off-by: Doug Hellmann <doug@doughellmann.com>
There are no logic changes in this patch, just changes to code
location, doc strings, and output messages.
Change-Id: I3fee69925337df2ad7870d97b393eb4837268ea3
Signed-off-by: Doug Hellmann <doug@doughellmann.com>
We check the type value via the schema now, so we don't need the list
here.
Change-Id: I349a24078bbca02d011d48c1adafcf9e4e13548e
Signed-off-by: Doug Hellmann <doug@doughellmann.com>
Rather than testing the series condition outside of the validation
function, go ahead and call it and let the function skip the
deliverable if its rules do not apply.
Change-Id: Ib02cdec2cba088bd13ba307822f0ecbb14ddc899
Signed-off-by: Doug Hellmann <doug@doughellmann.com>
This makes the signatures of all of the validation functions the same.
Change-Id: I97c5fc9422055aa46803424e4e05bc16e7cc289c
Signed-off-by: Doug Hellmann <doug@doughellmann.com>
The last checks being done by validate_releases are all related to
ensuring that one release in a series is on a commit that comes after
another commit in the same series' branch. Rename the function to
reflect that.
Change-Id: I05161eb6244da0fbd0e973d046c34ee93f8fcfa0
Signed-off-by: Doug Hellmann <doug@doughellmann.com>
We check the repositories against the repository-settings now so we
don't need to keep track of the changes in the contents of the
deliverable over time.
Change-Id: Ie5e7c42734eff252c36252610e151fcea1139ae7
Signed-off-by: Doug Hellmann <doug@doughellmann.com>
We were checking that 'untagged' deliverables didn't have releases in
the validate_releases function instead of in validate_model.
Change-Id: I0fcfecb6163fa45be79d9b24570b60f038e3c0a0
Signed-off-by: Doug Hellmann <doug@doughellmann.com>