feat: Define license assign endpoint on Gateway #255
Conversation
sameenfatima78
force-pushed
the
sameen/ENT-7995
branch
from
328a397
to
e58b957
Compare
fixed failing tests added tests
sameenfatima78
force-pushed
the
sameen/ENT-7995
branch
from
e58b957
to
cfd931a
Compare
| @@ -47,6 +47,8 @@ | |||
| help="Location of enterprise catalog IDA for request routing") | |||
| parser.add_argument('--authoring-host', required=True, | |||
| help="Location of Studio for authoring request routing") | |||
| parser.add_argument('--license-manager-host', required=True, | |||
There was a problem hiding this comment.
For few args like --enterprise-host it is marked as required=False, how we are figuring this out?
There was a problem hiding this comment.
That's a good catch but I guess it's ok to mark it required as long as the stage variables are being served here https://github.com/edx/edx-internal/pull/9905
|
@sameenfatima78 Good day! Could you please have a look at the build status for the I'm interested in merging unrelated changes, so please let me know if I can be of assistance, or, if that's not needed, when I should try my merge. |
|
@bszabo Hi, Bernard. I'm currently looking into this but haven't made much progress yet. Will let you know if I'm able to figure out the cause otherwise I can revert my changes to unblock you, if required. |
|
@bszabo Just pushed a fix and the pipeline is all green now. |
This PR adds a definition for license-manager assign endpoint on the Omnibus API to expose it to Gateway.
Host variable is also added to pick up host value for the service. https://github.com/edx/edx-internal/pull/9905
JIRA: https://2u-internal.atlassian.net/browse/ENT-7995