Opened 5 years ago

Closed 5 years ago

#1292 closed (fixed)

ION/MAX AMs have wrong authority in sliver URN

Reported by: Aaron Helsinger Owned by: xyang@maxgigapop.net
Priority: major Milestone:
Component: I2AM Version: SPIRAL6
Keywords: Cc: brecht.vermeulen@UGent.be, tlehman@maxgigapop.net
Dependencies:

Description

Brecht discovered that the authority portion of sliver URNs from the ION (and therefore also MAX) AMs is the authority of the _slice_, where it should be the authority of the _AM_.

A sample sliver status showing the error:

              "geni_urn": "urn:publicid:IDN+ch.geni.net:CHtest+slice+st3",
              "geni_resources": [
                     {
                        "geni_urn": "urn:publicid:IDN+ch.geni.net:CHtest+sliver+st3_vlan_ion.internet2.edu-116301",
                        "geni_error": "",
                        "geni_status": "ready"
                        }
                     ],

The authority portion of the sliver URN should be that of the aggregate, so ion.internet2.edu The sliver name should be unique within the aggregate. In this case it names the circuit ID and aggregate and slice name, so that should be sufficient. Alternatively you could add the slice authority name there as part of the sliver name, making the full name: ch.geni.net.st3_vlan_ion.internet2.edu-116301 or some such.

Change History (3)

comment:1 Changed 5 years ago by xyang@maxgigapop.net

Resolution: fixed
Status: newclosed

A patch has been made into code repository and applied to MAX AM.

We have to wait for the next ION AM upgrade to apply this to ION.

comment:2 Changed 5 years ago by Aaron Helsinger

Resolution: fixed
Status: closedreopened

Note that the sliver_urns in the manifest and sliverstatus are different. A sample urn from the manifest: urn:publicid:IDN+dragon.maxgigapop.net+sliver+dragon.maxgigapop.net-4039

And a sample from sliverstatus: urn:publicid:IDN+dragon.maxgigapop.net+sliver+ahtest_vlan_dragon.maxgigapop.net-4039

These should be the same.

This isn't critical, but it is more correct.

comment:3 Changed 5 years ago by xyang@maxgigapop.net

Resolution: fixed
Status: reopenedclosed

Fixed. The manifest will use the same format as sliverstatus does for sliver urn. The MAX AM has been patched. The patch will make into ION AM in next upgrade (within a week).

Note: See TracTickets for help on using tickets.