{"action":"created","issue":{"url":"https://api.github.com/repos/ietf-mpls-yang/te/issues/6","repository_url":"https://api.github.com/repos/ietf-mpls-yang/te","labels_url":"https://api.github.com/repos/ietf-mpls-yang/te/issues/6/labels{/name}","comments_url":"https://api.github.com/repos/ietf-mpls-yang/te/issues/6/comments","events_url":"https://api.github.com/repos/ietf-mpls-yang/te/issues/6/events","html_url":"https://github.com/ietf-mpls-yang/te/issues/6","id":267612761,"number":6,"title":"Use of topology-id","user":{"login":"italobusi","id":12461741,"avatar_url":"https://avatars0.githubusercontent.com/u/12461741?v=4","gravatar_id":"","url":"https://api.github.com/users/italobusi","html_url":"https://github.com/italobusi","followers_url":"https://api.github.com/users/italobusi/followers","following_url":"https://api.github.com/users/italobusi/following{/other_user}","gists_url":"https://api.github.com/users/italobusi/gists{/gist_id}","starred_url":"https://api.github.com/users/italobusi/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/italobusi/subscriptions","organizations_url":"https://api.github.com/users/italobusi/orgs","repos_url":"https://api.github.com/users/italobusi/repos","events_url":"https://api.github.com/users/italobusi/events{/privacy}","received_events_url":"https://api.github.com/users/italobusi/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":1,"created_at":"2017-10-23T10:13:25Z","updated_at":"2018-03-04T21:44:27Z","closed_at":"2018-03-04T21:44:27Z","author_association":"NONE","body":"The topology-id is used to identify  in which topology the tunnel is computed:\r\n\r\nhttps://github.com/ietf-mpls-yang/te/blob/b713c88398ef2594a637586f8f44418bfd3396d3/ietf-te.yang#L1348\r\n\r\nHowever, it seems that a topology is uniquely identified using provider-id, client-id and topology-id:\r\nhttps://tools.ietf.org/html/draft-ietf-teas-yang-te-topo-12#section-5.4\r\n\r\nI am wondering whether it is not worthwhile defining the following grouping in ietf-te-types and use it here:\r\n\r\n```\r\n\tgrouping te-topology-identifier {\r\n\t\tleaf provider-id {\r\n\t\t\ttype te-types:te-global-id;\r\n\t\t\tdescription\r\n\t\t\t\t\"An identifier to uniquely identify a provider.\";\r\n\t\t}\r\n\t\tleaf access-client-id {\r\n\t\t\ttype te-types:te-global-id;\r\n\t\t\tdescription\r\n\t\t\t\t\"An identifier to uniquely identify a client.\";\r\n\t\t}\r\n\t\tleaf access-topology-id {\r\n\t\t\ttype te-types:te-topology-id;\r\n\t\t\tdescription\r\n\t\t\t\t\"Identifies the topology.\";\r\n\t\t}\r\n\t}\r\n\r\n```\r\n\r\n"},"comment":{"url":"https://api.github.com/repos/ietf-mpls-yang/te/issues/comments/370266913","html_url":"https://github.com/ietf-mpls-yang/te/issues/6#issuecomment-370266913","issue_url":"https://api.github.com/repos/ietf-mpls-yang/te/issues/6","id":370266913,"user":{"login":"ietf-mpls-yang","id":9702603,"avatar_url":"https://avatars0.githubusercontent.com/u/9702603?v=4","gravatar_id":"","url":"https://api.github.com/users/ietf-mpls-yang","html_url":"https://github.com/ietf-mpls-yang","followers_url":"https://api.github.com/users/ietf-mpls-yang/followers","following_url":"https://api.github.com/users/ietf-mpls-yang/following{/other_user}","gists_url":"https://api.github.com/users/ietf-mpls-yang/gists{/gist_id}","starred_url":"https://api.github.com/users/ietf-mpls-yang/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ietf-mpls-yang/subscriptions","organizations_url":"https://api.github.com/users/ietf-mpls-yang/orgs","repos_url":"https://api.github.com/users/ietf-mpls-yang/repos","events_url":"https://api.github.com/users/ietf-mpls-yang/events{/privacy}","received_events_url":"https://api.github.com/users/ietf-mpls-yang/received_events","type":"User","site_admin":false},"created_at":"2018-03-04T21:44:27Z","updated_at":"2018-03-04T21:44:27Z","author_association":"OWNER","body":"Added to te-types module"}}
{"actor":{"display_login":"ietf-mpls-yang"}}