Age | Commit message (Collapse) | Author | Files | Lines |
|
This change should speed up group creation. Currently it's too slow,
especially for submissions with a lot of rebuilt packages.
Fixes: #1955
Change-Id: Id271e1bb30897bbb3807ac4221acef32d4d3eed1
Signed-off-by: Ed Bartosh <eduard.bartosh@intel.com>
|
|
This change should speed up group creation. Currently it's too slow,
especially for submissions with a lot of rebuilt packages.
Fixes: #1955
Change-Id: Id271e1bb30897bbb3807ac4221acef32d4d3eed1
Signed-off-by: Ed Bartosh <eduard.bartosh@intel.com>
|
|
It's hard for users to create the group with many submissions as
every time conflict is found group creation fails and they have
to start grouping again, removing conflicting submission.
This change makes repa to skip conflicting submission and continue
group creation without it. It should simplify and spead up RE work a lot.
Change-Id: I5a08059849e63e7fb8a07e4b05f9c81846d8af57
Signed-off-by: Ed Bartosh <eduard.bartosh@intel.com>
|
|
Change-Id: I11782fa592cbc11740dfe9b2ae62520d1b810c9b
Signed-off-by: Ed Bartosh <eduard.bartosh@intel.com>
|
|
If publishing is enabled OBS publishes repository after each
aggregate operation, which triggers a lot of not needed image
creation and testing events. It can cause wrong test results and
accepting of bad submissions.
Change-Id: Idf6fd28101feb7857fca4b7830acd797e2b4d487
Signed-off-by: Ed Bartosh <eduard.bartosh@intel.com>
|
|
Change-Id: Icb7ecc36649eaa580b6ac7a91cf9a16a9955820e
Signed-off-by: Ed Bartosh <eduard.bartosh@intel.com>
|
|
Change-Id: I787a06d2b1bbb88bcd1a943657759ad0e89f9228
Signed-off-by: Ed Bartosh <eduard.bartosh@intel.com>
|
|
Stopped using OpenSUSE format in favor of using git-rpm-ch
to maintain changelogs.
Change-Id: Ifd0992b7e74aa348c5f2ca91b28bab171826f473
Signed-off-by: Ed Bartosh <eduard.bartosh@intel.com>
|
|
Fixed wrong description of accept and reject. They don't accept
list of submissions and submit groups. They accept only one parameter -
submission or submit group.
Change-Id: Ib4add5cbe027e02638a742cd156c1c5d84aceea3
Signed-off-by: Ed Bartosh <eduard.bartosh@intel.com>
|
|
repa diff shows the difference between manifest of target project and
another (usually base) project. It also shows latest accepted submission
in another project and its status in target project.
repa diff should help RE to mantain the difference between base
and target projects.
Change-Id: I6ef93a5eb2c5a60b3778a721caa8be47da9f5882
Signed-off-by: Ed Bartosh <eduard.bartosh@intel.com>
|
|
Although it's tiny API it makes sense to implement it as
it keeps usage of OBS_PROJECT_PREFIX inside common.py
Change-Id: Ib5166f4804499576f8540a3141129507410beb49
Signed-off-by: Ed Bartosh <eduard.bartosh@intel.com>
|
|
Separated get_project_list API from get_project as it will be called
by repa diff and other higher level code.
git_project performs a lot more queries then just getting list of
projects as it als gets build results and descriptions for every
project. git_project_list only gets the list of projects, which is
much faster.
Change-Id: I285b19b478d384cd4473ba2b8a8ef9637c611538
Signed-off-by: Ed Bartosh <eduard.bartosh@intel.com>
|
|
To distinguish groups from simple prerelease projects let's add
-group suffix to project path.
This was requested by QA team as they may consider to test only
group images as they don't have enough capacity to test all submissions,
so they need a way to distinguish groups.
Change-Id: I6c5f6d87d5e6b7ee1a51c84407876b8513111fa8
Signed-off-by: Ed Bartosh <eduard.bartosh@intel.com>
|
|
Implemented get_srs to get SRs from the project.
Optionally SRs can be filtered by package and comma-separated
list of statuses.
Change-Id: If003f25e1d54742e13cd1c1d1418598ee4baea15
Signed-off-by: Ed Bartosh <eduard.bartosh@intel.com>
|
|
Currently repa creates one SR per package.
This change implements creation of one SR per multiple packages by using new
schema of POST /request API https://api.opensuse.org/apidocs/request.rng
Note, that osc API doesn't support this schema, so it can't be used.
Fixes: #1386
Change-Id: Iae5cb55b0b5f489b9596c5a79e0845ec18a2ba51
Signed-off-by: Ed Bartosh <eduard.bartosh@intel.com>
|
|
When repa shows build failures it should not show packages in 'building'
and 'blocked' statuses.
Change-Id: I1ad2af336c9499105d869104e45d83afcbc8e903
Signed-off-by: Ed Bartosh <eduard.bartosh@intel.com>
|
|
This check should not allow to create group from submissions with common
binary packages. It didn't work because of the incomplete implementation.
Fixes: #1847
Change-Id: I7e756913f8756bdb330bf31dae9574e973717f6c
Signed-off-by: Ed Bartosh <eduard.bartosh@intel.com>
|
|
This status will be shown when there is no information about images in meta.
Change-Id: I0263f9860fccf400ba36bf1d77840d0d006692f6
Signed-off-by: Ed Bartosh <eduard.bartosh@intel.com>
|
|
Blocked status was not processed before, which caused incorrect
status reporting by repa list.
Change-Id: Ibf219d914ef8cc3dc2a7d30881dbb6520c0fc19e
Signed-off-by: Ed Bartosh <eduard.bartosh@intel.com>
|
|
Some users want to see urls in repa list output. It's inconvenient for
them to use repa info for every submission to see the urls.
Fixes: #1789
Change-Id: I7cc31bc327fc9593ab21efdd85fe4fe6accfd544
Signed-off-by: Ed Bartosh <eduard.bartosh@intel.com>
|
|
Make status field wider as some statuses don't feet.
Change-Id: Ia79d9dae30b76f669bff57d556e201c4e28ffd52
Signed-off-by: Ed Bartosh <eduard.bartosh@intel.com>
|
|
For now only status is colorized in repa list output. More colorized
pieces will come if users request them.
Fixes: #1765
Change-Id: Ic0517aaa03f443d4e661018a9638712a9afa514b
Signed-off-by: Ed Bartosh <eduard.bartosh@intel.com>
|
|
This is going to be used to colorize repa output.
Change-Id: I5d9eddc427c80b787b4fd5d05a46e08b624aa4cc
Signed-off-by: Ed Bartosh <eduard.bartosh@intel.com>
|
|
Dropped --regexp in favor of --project, which is much more clear and
used in all modes.
Fixes: #1772
Change-Id: I5347190f41f6c704fe2091909435aaeaa5452f62
Signed-off-by: Ed Bartosh <eduard.bartosh@intel.com>
|
|
Change-Id: Ia5f5e02b9231d11c85b884be9c7c64c64bbfed11
Signed-off-by: Ed Bartosh <eduard.bartosh@intel.com>
|
|
repa list was showing overall status as 'Ready' for submissions with
unresolvable packages.
Fixes: #1776
Change-Id: I5ca5e0cc481329de8aa487cd212a66a29f005b94
Signed-off-by: Ed Bartosh <eduard.bartosh@intel.com>
|
|
Mentioned <global options> in synopsys.
Renamed section OPTIONS->GLOBAL OPTIONS
Change-Id: Iac09199756ef0d1e9d24a851be0266f6a9a370b6
Signed-off-by: Ed Bartosh <eduard.bartosh@intel.com>
|
|
Change-Id: Ie9d3aca2ca3f80fa0f24fa84b7dcb7d3896d42b8
Signed-off-by: Ed Bartosh <eduard.bartosh@intel.com>
|
|
Global command line option -s/--section was not mentioned in the man.
Change-Id: I5b765258fb2d3b76e5dfcecb22aae69262806b25
Signed-off-by: Ed Bartosh <eduard.bartosh@intel.com>
|
|
Fixes: #1764
Change-Id: I62d75e44aac796605c23cfc5ad9820e44b55de9d
Signed-off-by: Ed Bartosh <eduard.bartosh@intel.com>
|
|
As --project option was mandatory in most of the subcommands let's make
project mandatory config option.
--project becomes non-mandatory global option.
Fixes: #1763
Change-Id: Id7f94fc5d60a1d163bde614ecf0605f944448337
Signed-off-by: Ed Bartosh <eduard.bartosh@intel.com>
|
|
repa info shows as much information about one submission as possible.
Fixes: #1195
Change-Id: I3822bae22c0f86abdaa842c067c95ade6f99d918
Signed-off-by: Ed Bartosh <eduard.bartosh@intel.com>
|
|
Repa info will print only small part of information available.
More detailed output will be implemented in repa info mode.
Change-Id: I8e7ba14588f7e550515f0c03cfc84051ee2d962c
Signed-off-by: Ed Bartosh <eduard.bartosh@intel.com>
|
|
get_projects starts to fail to query OBS more frequently in parallel
mode. Using nice implementation of retry decorator to repeat queries for
2 methods running under multiprocessing pool.
Change-Id: I175a528d9c0d7aa2951c3b659a1f5f85a8ae614f
Signed-off-by: Ed Bartosh <eduard.bartosh@intel.com>
|
|
Repa list makes a lot of queries to OBS, which results in slow output.
With --processes it queries OBS in parallel utilizing functionality of
multiprocessing module.
This is an experimental feature. Hopefully it will go away when repa
will start using IRIS ReST API.
Fixes: #1762
Change-Id: I47424e5e5c044dd75589f0d0dcb01a7bbff068e8
Signed-off-by: Ed Bartosh <eduard.bartosh@intel.com>
|
|
Change-Id: I17b1b12a66bc34a0b725b0e38644ef14b6f8b55d
Signed-off-by: Ed Bartosh <eduard.bartosh@intel.com>
|
|
Fixes this build failure:
File not found by glob: .../site-packages//repa-0.2-*.egg-info
File not found by glob: .../site-packages/repa-0.2-*-nspkg.pth
Change-Id: I5ea703220aa030773bfb598e12d4b882b6e8167a
Signed-off-by: Ed Bartosh <eduard.bartosh@intel.com>
|
|
Change-Id: I3199ac4105725cf575dd0569759537ec42ebace0
Signed-off-by: Ed Bartosh <eduard.bartosh@intel.com>
|
|
By default repa now checks if binary packages exist in submission and
doesn't allow grouping submissions without binary packages.
--force option allows to bypass this check, i.e. allows to group failed
submissions. This can be useful for grouping failed submissions to
reject them as a group.
Fixes: #1707
Change-Id: Ieb66718839e19b63de5039757a2bbc63d02016c6
Signed-off-by: Ed Bartosh <eduard.bartosh@intel.com>
|
|
Change-Id: I67145fe75644fc6c18a73a6ec2c6676aee13c094
Signed-off-by: Ed Bartosh <eduard.bartosh@intel.com>
Reviewed-on: https://otctools.jf.intel.com/review/10690
Tested-by: OTC Tools Tester <ed.bartosh@linux.intel.com>
Reviewed-by: Markus Lehtonen <markus.lehtonen@linux.intel.com>
|
|
RELEASE_NOTES file is required to be included to the package by Tools
release manager. They're automatically processed when release is
published and added to the Tools release notes.
Change-Id: Ifb392924c0e25aa845df6d1095ee4b4e5598a932
Signed-off-by: Ed Bartosh <eduard.bartosh@intel.com>
Reviewed-on: https://otctools.jf.intel.com/review/10687
Tested-by: OTC Tools Tester <ed.bartosh@linux.intel.com>
Reviewed-by: Markus Lehtonen <markus.lehtonen@linux.intel.com>
Reviewed-by: Jian-feng Ding <jian-feng.ding@intel.com>
|
|
To obtain revision create_sr code queries source package and crashes
when source is broken with ObsError: <obs>cat't query source <package>
from project <prerelease project>: HTTP Error 400: service gbs failed
This makes it impossible to reject broken submissions.
This change catches exeption and tries to avoid querying the source by
providing revision 1 to create_sr API. As for rejecting it doesn't
matter much which revision to reject it should be safe.
Fixes: #1687
Change-Id: I21b9dda32686c94bfb8e81201424b3e7e7f128b5
Signed-off-by: Ed Bartosh <eduard.bartosh@intel.com>
Reviewed-on: https://otctools.jf.intel.com/review/10674
Tested-by: OTC Tools Tester <ed.bartosh@linux.intel.com>
|
|
It's better to raise RepaException as it is caught in main.py and error
is printed on stderr. ObsError produces ugly python traceback and scares
users.
References: #1687
Change-Id: Iea85eb263c7e4010df1fa50bf6e16da1ef7f1af0
Signed-off-by: Ed Bartosh <eduard.bartosh@intel.com>
Reviewed-on: https://otctools.jf.intel.com/review/10673
Tested-by: OTC Tools Tester <ed.bartosh@linux.intel.com>
Reviewed-by: Markus Lehtonen <markus.lehtonen@linux.intel.com>
|
|
Catch OSCError exceptions and raises RepaException with the meaningful
error message. RepaException is processed in main.py and prints error
message to stderr.
Fixes: #1708
Change-Id: I966013766e2756d64edf02c9cc6ac51d42f3a525
Signed-off-by: Ed Bartosh <eduard.bartosh@intel.com>
Reviewed-on: https://otctools.jf.intel.com/review/10671
Tested-by: OTC Tools Tester <ed.bartosh@linux.intel.com>
|
|
This is done due to refactoring purposes to make it possible for obs.py
to import common module.
Change-Id: I511a9ea57e320930a93bb5c1b45306287e832a59
Signed-off-by: Ed Bartosh <eduard.bartosh@intel.com>
Reviewed-on: https://otctools.jf.intel.com/review/10670
Tested-by: OTC Tools Tester <ed.bartosh@linux.intel.com>
Reviewed-by: Markus Lehtonen <markus.lehtonen@linux.intel.com>
|
|
Submitter will be added to SR commit If 'submitter' key is present
in meta. This key should be added by backend.
Fixes: #1723
Change-Id: I731df88a247962467b849b9572b4ab9f85077d44
Signed-off-by: Ed Bartosh <eduard.bartosh@intel.com>
Reviewed-on: https://otctools.jf.intel.com/review/10666
Tested-by: OTC Tools Tester <ed.bartosh@linux.intel.com>
Reviewed-by: Markus Lehtonen <markus.lehtonen@linux.intel.com>
|
|
Current implementation puts git tag as a commit id into the SR comment.
This change will make repa to use git_commit from the meta if it's
present there. Backend should be also modified to put git_commit into
meta: https://otctools.jf.intel.com/pm/issues/1732
Fixes: #1709
Change-Id: Iae01404652160fb4d86a1c0236e226b28773d225
Signed-off-by: Ed Bartosh <eduard.bartosh@intel.com>
Reviewed-on: https://otctools.jf.intel.com/review/10664
Tested-by: OTC Tools Tester <ed.bartosh@linux.intel.com>
Reviewed-by: Markus Lehtonen <markus.lehtonen@linux.intel.com>
|
|
Added notice explaining the purpose(image testing) and details(binary
aggregation) of implementation of groups.
Fixes: #TINF-492
Change-Id: Id5b71891c9ffffe9b962637ed7c65442550930bc
Signed-off-by: Ed Bartosh <eduard.bartosh@intel.com>
Reviewed-on: https://otctools.jf.intel.com/review/10469
Tested-by: OTC Tools Tester <ed.bartosh@linux.intel.com>
Reviewed-by: Markus Lehtonen <markus.lehtonen@linux.intel.com>
|
|
Change-Id: I919bfe46b4023bc344927a57040f3c24e294a07c
Signed-off-by: Ed Bartosh <eduard.bartosh@intel.com>
Reviewed-on: https://otctools.jf.intel.com/review/10079
Reviewed-by: Markus Lehtonen <markus.lehtonen@linux.intel.com>
Tested-by: OTC Tools Tester <ed.bartosh@linux.intel.com>
|
|
Repa gets both urls from correspondent fields in meta. If they aren't
present in meta repa tries to guess them. Guessing part is temporary.
It will be removed when backend provides urls.
Fixes: #1674
Change-Id: I9f8f63572ae3407b08f1e9d0f7028d237f4fc1d9
Signed-off-by: Ed Bartosh <eduard.bartosh@intel.com>
Reviewed-on: https://otctools.jf.intel.com/review/10078
Tested-by: OTC Tools Tester <ed.bartosh@linux.intel.com>
Reviewed-by: Markus Lehtonen <markus.lehtonen@linux.intel.com>
|