{"action":"created","issue":{"url":"https://api.github.com/repos/FRRouting/frr/issues/3327","repository_url":"https://api.github.com/repos/FRRouting/frr","labels_url":"https://api.github.com/repos/FRRouting/frr/issues/3327/labels{/name}","comments_url":"https://api.github.com/repos/FRRouting/frr/issues/3327/comments","events_url":"https://api.github.com/repos/FRRouting/frr/issues/3327/events","html_url":"https://github.com/FRRouting/frr/pull/3327","id":380519466,"node_id":"MDExOlB1bGxSZXF1ZXN0MjMwNjk4MDA5","number":3327,"title":"bgpd, zebra: auto assign labels to regular labeled-unicast prefixes","user":{"login":"adeg","id":3998049,"node_id":"MDQ6VXNlcjM5OTgwNDk=","avatar_url":"https://avatars1.githubusercontent.com/u/3998049?v=4","gravatar_id":"","url":"https://api.github.com/users/adeg","html_url":"https://github.com/adeg","followers_url":"https://api.github.com/users/adeg/followers","following_url":"https://api.github.com/users/adeg/following{/other_user}","gists_url":"https://api.github.com/users/adeg/gists{/gist_id}","starred_url":"https://api.github.com/users/adeg/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/adeg/subscriptions","organizations_url":"https://api.github.com/users/adeg/orgs","repos_url":"https://api.github.com/users/adeg/repos","events_url":"https://api.github.com/users/adeg/events{/privacy}","received_events_url":"https://api.github.com/users/adeg/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":{"login":"rwestphal","id":931662,"node_id":"MDQ6VXNlcjkzMTY2Mg==","avatar_url":"https://avatars1.githubusercontent.com/u/931662?v=4","gravatar_id":"","url":"https://api.github.com/users/rwestphal","html_url":"https://github.com/rwestphal","followers_url":"https://api.github.com/users/rwestphal/followers","following_url":"https://api.github.com/users/rwestphal/following{/other_user}","gists_url":"https://api.github.com/users/rwestphal/gists{/gist_id}","starred_url":"https://api.github.com/users/rwestphal/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rwestphal/subscriptions","organizations_url":"https://api.github.com/users/rwestphal/orgs","repos_url":"https://api.github.com/users/rwestphal/repos","events_url":"https://api.github.com/users/rwestphal/events{/privacy}","received_events_url":"https://api.github.com/users/rwestphal/received_events","type":"User","site_admin":false},"assignees":[{"login":"rwestphal","id":931662,"node_id":"MDQ6VXNlcjkzMTY2Mg==","avatar_url":"https://avatars1.githubusercontent.com/u/931662?v=4","gravatar_id":"","url":"https://api.github.com/users/rwestphal","html_url":"https://github.com/rwestphal","followers_url":"https://api.github.com/users/rwestphal/followers","following_url":"https://api.github.com/users/rwestphal/following{/other_user}","gists_url":"https://api.github.com/users/rwestphal/gists{/gist_id}","starred_url":"https://api.github.com/users/rwestphal/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rwestphal/subscriptions","organizations_url":"https://api.github.com/users/rwestphal/orgs","repos_url":"https://api.github.com/users/rwestphal/repos","events_url":"https://api.github.com/users/rwestphal/events{/privacy}","received_events_url":"https://api.github.com/users/rwestphal/received_events","type":"User","site_admin":false}],"milestone":null,"comments":5,"created_at":"2018-11-14T03:24:15Z","updated_at":"2018-11-14T17:45:26Z","closed_at":null,"author_association":"CONTRIBUTOR","pull_request":{"url":"https://api.github.com/repos/FRRouting/frr/pulls/3327","html_url":"https://github.com/FRRouting/frr/pull/3327","diff_url":"https://github.com/FRRouting/frr/pull/3327.diff","patch_url":"https://github.com/FRRouting/frr/pull/3327.patch"},"body":"### Summary\r\nThis is the last missing piece to full BGP LU support. To this moment, bgpd (and zebra) supported auto label assignment only for prefixes leaked from VRFs to vpn and for MPLS SR prefixes. This adds auto label assignment to other route types in bgpd. The following enhancements have been made:\r\n- bgp_route.c:bgp_process_main_one() now sets implicit-null local_label to all local, aggregate and redistributed routes.\r\n- bgp_label.c:bgp_red_dereg_for_label() now requests labels from label pool for routes which have no associated label index\r\n- zebra_mpls.c:zebra_mpls_fec_register() now expects both label and label_index from the calling function, one of which must be set to MPLS_INVALID_LABEL or MPLS_INVALID_LABEL_INDEX, based on this it will decide how to register the provided FEC.\r\n\r\n### Components\r\n[bgpd, zebra]"},"comment":{"url":"https://api.github.com/repos/FRRouting/frr/issues/comments/438753315","html_url":"https://github.com/FRRouting/frr/pull/3327#issuecomment-438753315","issue_url":"https://api.github.com/repos/FRRouting/frr/issues/3327","id":438753315,"node_id":"MDEyOklzc3VlQ29tbWVudDQzODc1MzMxNQ==","user":{"login":"NetDEF-CI","id":19297776,"node_id":"MDQ6VXNlcjE5Mjk3Nzc2","avatar_url":"https://avatars3.githubusercontent.com/u/19297776?v=4","gravatar_id":"","url":"https://api.github.com/users/NetDEF-CI","html_url":"https://github.com/NetDEF-CI","followers_url":"https://api.github.com/users/NetDEF-CI/followers","following_url":"https://api.github.com/users/NetDEF-CI/following{/other_user}","gists_url":"https://api.github.com/users/NetDEF-CI/gists{/gist_id}","starred_url":"https://api.github.com/users/NetDEF-CI/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/NetDEF-CI/subscriptions","organizations_url":"https://api.github.com/users/NetDEF-CI/orgs","repos_url":"https://api.github.com/users/NetDEF-CI/repos","events_url":"https://api.github.com/users/NetDEF-CI/events{/privacy}","received_events_url":"https://api.github.com/users/NetDEF-CI/received_events","type":"User","site_admin":false},"created_at":"2018-11-14T17:45:26Z","updated_at":"2018-11-14T17:45:26Z","author_association":"COLLABORATOR","body":"# Continuous Integration Result: SUCCESSFUL\n\nCongratulations, this patch passed basic tests\n\nTested-by: NetDEF / OpenSourceRouting.org CI System\n\nCI System Testrun URL: https://ci1.netdef.org/browse/FRR-FRRPULLREQ-5921/\n\nThis is a comment from an EXPERIMENTAL automated CI system.\nFor questions and feedback in regards to this CI system, please feel free to email\nMartin Winter - mwinter (at) opensourcerouting.org.\n\n\n-----------------------------\n## Warnings Generated during build:\n\n\n### Checkout code: Successful with additional warnings:\n\n    Report for bgp_label.c | 20 issues\n    ===============================================\n    < WARNING: quoted string split across lines\n    < #166: FILE: /tmp/f1-1552/bgp_label.c:166:\n    < WARNING: line over 80 characters\n    < #167: FILE: /tmp/f1-1552/bgp_label.c:167:\n    < WARNING: line over 80 characters\n    < #215: FILE: /tmp/f1-1552/bgp_label.c:215:\n    < WARNING: line over 80 characters\n    < #220: FILE: /tmp/f1-1552/bgp_label.c:220:\n    < WARNING: line over 80 characters\n    < #223: FILE: /tmp/f1-1552/bgp_label.c:223:\n    < WARNING: line over 80 characters\n    < #228: FILE: /tmp/f1-1552/bgp_label.c:228:\n    < WARNING: line over 80 characters\n    < #229: FILE: /tmp/f1-1552/bgp_label.c:229:\n    < WARNING: line over 80 characters\n    < #236: FILE: /tmp/f1-1552/bgp_label.c:236:\n    < WARNING: line over 80 characters\n    < #237: FILE: /tmp/f1-1552/bgp_label.c:237:\n    < WARNING: line over 80 characters\n    < #240: FILE: /tmp/f1-1552/bgp_label.c:240:\n    Report for bgp_route.c | 8 issues\n    ===============================================\n    < WARNING: line over 80 characters\n    < #2274: FILE: /tmp/f1-1552/bgp_route.c:2274:\n    < WARNING: line over 80 characters\n    < #2277: FILE: /tmp/f1-1552/bgp_route.c:2277:\n    < WARNING: line over 80 characters\n    < #2278: FILE: /tmp/f1-1552/bgp_route.c:2278:\n    < WARNING: line over 80 characters\n    < #2281: FILE: /tmp/f1-1552/bgp_route.c:2281:\n    Report for zebra_mpls.c | 6 issues\n    ===============================================\n    < WARNING: line over 80 characters\n    < #1821: FILE: /tmp/f1-1552/zebra_mpls.c:1821:\n    < WARNING: quoted string split across lines\n    < #1843: FILE: /tmp/f1-1552/zebra_mpls.c:1843:\n    < WARNING: line over 80 characters\n    < #1844: FILE: /tmp/f1-1552/zebra_mpls.c:1844:\n    Report for zebra_mpls.h | 2 issues\n    ===============================================\n    < WARNING: line over 80 characters\n    < #203: FILE: /tmp/f1-1552/zebra_mpls.h:203:\n\n\n-----------------------------\n### CLANG Static Analyzer Summary\n- Github Pull Request 3327, comparing to Git base SHA 9787eeb\n#### No Changes in Static Analysis warnings compared to base\n"}}
{"actor":{"display_login":"NetDEF-CI"}}