As a user I can upload comps.xml file. #709
Conversation
|
Refer to this link for build results (access rights to CI server needed): |
| @@ -376,6 +376,11 @@ Metadata | |||
| Example format: ``{"group" : <group_id>, "default" : True}`` | |||
|
|
|||
|
|
|||
| .. note:: | |||
| Package_group, package_category and package environment elements can be uploaded also via comps file. | |||
There was a problem hiding this comment.
s/can be uploaded also via/can also be uploaded via/
|
It looks fine to me, but I am pretty unfamiliar with this, someone else should also take a look. |
| from pulp_rpm.devel.client_base import PulpClientTests | ||
|
|
||
|
|
||
| DATA_DIR = os.path.abspath(os.path.dirname(__file__)) + '/../../../../../../plugins/test' |
There was a problem hiding this comment.
I've seen this elsewhere, but it doesn't seem ideal. @rbarlow, do you know of a better way?
There was a problem hiding this comment.
Austin wrote:
I've seen this elsewhere, but it doesn't seem ideal. @rbarlow
https://github.com/rbarlow, do you know of a better way?
Eh, for tests I don't think it's so bad. But one option could be to put
the DATA_DIR variable inside pulp_rpm.devel.somewhere and then you could
import that instead.
|
Fix these two things before merging:
LGTM! |
|
Refer to this link for build results (access rights to CI server needed): |
closes #287
https://pulp.plan.io/issues/287