Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

SERVICE_CERT is a local file path #580

Open
nbastin opened this issue Jul 14, 2017 · 2 comments
Open

SERVICE_CERT is a local file path #580

nbastin opened this issue Jul 14, 2017 · 2 comments

Comments

@nbastin
Copy link

nbastin commented Jul 14, 2017

The SERVICE_CERT member returned from lookup is a local path, not an actual cert:

           {'SERVICE_CERT': '/usr/share/geni-chapi/sr/certs/wvn-eg.pem',
            'SERVICE_DESCRIPTION': 'WVNET ExoGENI Rack',
            'SERVICE_NAME': 'WVNET ExoGENI',
            'SERVICE_TYPE': 0,
            'SERVICE_URL': 'https://wvn-hn.exogeni.net:11443/orca/xmlrpc',
            'SERVICE_URN': 'urn:publicid:IDN+exogeni.net:wvnvmsite+authority+am',
            '_GENI_SERVICE_ATTRIBUTES': {'SPEAKS_FOR': 't',
                                         'UI_AM_CAT': 'ui_experimental_cat ui_compute_cat',
                                         'UI_AM_TYPE': 'ui_exogeni_am'},
            '_GENI_SERVICE_CERT_FILENAME': '/usr/share/geni-chapi/sr/certs/wvn-eg.pem',
            '_GENI_SERVICE_ID': 417,
            '_GENI_SERVICE_SHORT_NAME': 'wvn-eg’},
@tcmitchell
Copy link
Member

The service registry no longer requires certificates, and the recently added aggregates won't have a certificate, file name or otherwise. The certificates were required to support Flack, which was dropped a couple of years ago.

We should either remove the 'SERVICE_CERT' field from the return value for aggregates or we should change its value to an empty string.

@tcmitchell
Copy link
Member

Oddly, there is code to fill in the contents of the certificate file. Why isn't that working? Perhaps this code should be resurrected, although as noted above, not all aggregates will have certificate information.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants