{"push_id":1193258675,"size":80,"distinct_size":2,"ref":"refs/heads/volatile/patch-tracking/8/proposed/nits/a","head":"59d26cbc56bf8074ca6c87e4268eace87da57c14","before":"b707423d3b85ae69291dfaef5f786b6c75192df7","commits":[{"sha":"6bbd2d58e208920b86d41649f8ce67499576ec56","author":{"name":"Donald Sharp","email":"5fd60869f3646fa4d7b238697c8dacce9d2249af@cumulusnetworks.com"},"message":"*: Consolidate all double VIEW_NODE and ENABLE_NODE's\n\nIf a command is put into the VIEW_NODE, it is going into the\nENABLE_NODE as well.  This is especially true for show commands.\nAs such if a command is in both consolidate it down to VIEW_NODE.\n\nSigned-off-by: Donald Sharp <sharpd@cumulusnetworks.com>","distinct":false,"url":"https://api.github.com/repos/opensourcerouting/quagga/commits/6bbd2d58e208920b86d41649f8ce67499576ec56"},{"sha":"aa8e8079f5fa5c9057a600e45ace911207389317","author":{"name":"David Lamparter","email":"e7508b0a1c50eacfe329c28bd3d5eee8293d5793@opensourcerouting.org"},"message":"lib: fix MIN/MAX macros to not double-eval\n\ncf. https://gcc.gnu.org/onlinedocs/gcc/Typeof.html\n(Works on all compilers on Quagga's compiler support list in\ndoc/overview.texi)\n\nSigned-off-by: David Lamparter <equinox@opensourcerouting.org>\nTested-by: NetDEF CI System <cisystem@netdef.org>\nAcked-by: Donald Sharp <sharpd@cumulusnetworks.com>","distinct":false,"url":"https://api.github.com/repos/opensourcerouting/quagga/commits/aa8e8079f5fa5c9057a600e45ace911207389317"},{"sha":"a7ab6ef7b3d5a7098523ecacfe92e24c4ea55edc","author":{"name":"Donald Sharp","email":"5fd60869f3646fa4d7b238697c8dacce9d2249af@cumulusnetworks.com"},"message":"lib: Abstract fd set operations\n\nAbstract FD set operations so that we can eventually\nchoose what type of select/poll operation that\nwe want to use.\n\nSigned-off-by: Donald Sharp <sharpd@cumulusnetowkrs.com>","distinct":false,"url":"https://api.github.com/repos/opensourcerouting/quagga/commits/a7ab6ef7b3d5a7098523ecacfe92e24c4ea55edc"},{"sha":"3002c391573d654b33002552edacca3d27f8ff2b","author":{"name":"Donald Sharp","email":"5fd60869f3646fa4d7b238697c8dacce9d2249af@cumulusnetworks.com"},"message":"lib: Refactor read/write functionality\n\nBoth the read and write functions used the same code\nslightly modified for reading and writing.  Combine this\ncode together.\n\nSigned-off-by: Donald Sharp <sharpd@cumulusnetworks.com>\n\nEdited-by: Paul Jakma <paul.jakma@hpe.com> to retain the\nexternal library symbols, for ease of merging.","distinct":false,"url":"https://api.github.com/repos/opensourcerouting/quagga/commits/3002c391573d654b33002552edacca3d27f8ff2b"},{"sha":"fe631bcc25ed682296bcdc07cbfc3f7f960d19cf","author":{"name":"Donald Sharp","email":"5fd60869f3646fa4d7b238697c8dacce9d2249af@cumulusnetworks.com"},"message":"lib: Refactor thread_process_fd\n\nthread_process_fd is looping over the read and write\nfd's separately.  There is no need to do this individually.\nloop over both the read and write fdset's at the same time.\nThis will improve select processing performance, especially\nfor large data sets.\n\nSigned-off-by: Donald Sharp <sharpd@cumulusnetworks.com>","distinct":false,"url":"https://api.github.com/repos/opensourcerouting/quagga/commits/fe631bcc25ed682296bcdc07cbfc3f7f960d19cf"},{"sha":"839ac2874592248d77e1294584e429e6cacba058","author":{"name":"Donald Sharp","email":"5fd60869f3646fa4d7b238697c8dacce9d2249af@cumulusnetworks.com"},"message":"bgpd: Fix code path that leads to uninitialized variables\n\nsubtype and sublength are not initialized and if on Line 1877\nBGP_ATTR_ENCAP != type we will not set subtype and sublength,\nbut these variables are used immediately below that if statement.\n\nThis issue was discovered via the freebsd compiler.\n\nSigned-off-by: Donald Sharp <sharpd@cumulusnetworks.com>","distinct":false,"url":"https://api.github.com/repos/opensourcerouting/quagga/commits/839ac2874592248d77e1294584e429e6cacba058"},{"sha":"eeb11a01dc8b41e8497a200c8cba88cabd1e67f7","author":{"name":"Donald Sharp","email":"5fd60869f3646fa4d7b238697c8dacce9d2249af@cumulusnetworks.com"},"message":"lib: Remove unnecessary parentheses\n\nThe freebsd compiler complains that there is an extra set of ()'s\naround the if statement.\n\nSigned-off-by: Donald Sharp <sharpd@cumulusnetworks.com>","distinct":false,"url":"https://api.github.com/repos/opensourcerouting/quagga/commits/eeb11a01dc8b41e8497a200c8cba88cabd1e67f7"},{"sha":"26a88501fc403c9a6f828a899d88c2f4171af926","author":{"name":"Pawel Wieczorkiewicz","email":"f96067564f322764ac5dc538202aba1cee650f54@suse.de"},"message":"build: detect and create AM_SILENT_RULES macro if needed\n\nOlder libs have problem with that:\nconfigure.ac:17: warning: macro `AM_SILENT_RULES'\n  not found in library\n...\nconfigure.ac:24: error: possibly undefined macro:\n   AM_SILENT_RULES\nTested-by: NetDEF CI System <cisystem@netdef.org>","distinct":false,"url":"https://api.github.com/repos/opensourcerouting/quagga/commits/26a88501fc403c9a6f828a899d88c2f4171af926"},{"sha":"e09d7f89dc53d44dd5843c13875b8f1025183397","author":{"name":"Christian Franke","email":"ce7ac637ac48b67a36141bd2a33b2fe771398b96@nowhere.ws"},"message":"lib: use constant to replace magic value for length of quagga_timestamp\n\nSigned-off-by: Christian Franke <chris@opensourcerouting.org>","distinct":false,"url":"https://api.github.com/repos/opensourcerouting/quagga/commits/e09d7f89dc53d44dd5843c13875b8f1025183397"},{"sha":"7592d95927ca6a2a97e37d131473b39a58fd1d23","author":{"name":"Christian Franke","email":"ce7ac637ac48b67a36141bd2a33b2fe771398b96@nowhere.ws"},"message":"lib: add api method to read current config path\n\nSigned-off-by: Christian Franke <chris@opensourcerouting.org>","distinct":false,"url":"https://api.github.com/repos/opensourcerouting/quagga/commits/7592d95927ca6a2a97e37d131473b39a58fd1d23"},{"sha":"61604cef969136026ad218dc88d5ed78229be6da","author":{"name":"Christian Franke","email":"ce7ac637ac48b67a36141bd2a33b2fe771398b96@nowhere.ws"},"message":"lib: add event_counter\n\nSigned-off-by: Christian Franke <chris@opensourcerouting.org>","distinct":false,"url":"https://api.github.com/repos/opensourcerouting/quagga/commits/61604cef969136026ad218dc88d5ed78229be6da"},{"sha":"66c62e95460d84c7e942b08a315d6458ad5d018b","author":{"name":"Christian Franke","email":"ce7ac637ac48b67a36141bd2a33b2fe771398b96@nowhere.ws"},"message":"zebra: count iface up/down events and keep last time of their occurrence\n\nIt is quite useful to be able to assert whether specific interfaces have\nflapped or also to verify that specific interfaces have not flapped.\n\nBy having counters for those events and storing the last time of their\noccurrence, this is made possible.\n\nSigned-off-by: Christian Franke <chris@opensourcerouting.org>\nTested-by: NetDEF CI System <cisystem@netdef.org>","distinct":false,"url":"https://api.github.com/repos/opensourcerouting/quagga/commits/66c62e95460d84c7e942b08a315d6458ad5d018b"},{"sha":"e226f00e96d5b3e03195d7f1c1d476453235916b","author":{"name":"Paul Jakma","email":"54f65660711c038b3360dd7288dc12347ae58ba3@opensourcerouting.org"},"message":"lib/stream: add float/double <-> IEEE-754 single/double mux/demuxers\n\n* stream.{c,h}: Add stream_get{f,d} and stream_put{f,d}) demux and muxers to\n  safely convert between big-endian IEEE-754 single and double binary\n  format, as used in IETF RFCs, and C99.  Implementation depends on host\n  using __STDC_IEC_559__, which should be everything we care about.  Should\n  correctly error out otherwise.\n\n  Thanks to Aidan Delaney <aidan@phoric.eu> and Olivier Dugeon for pointing\n  out the __STDC_IEC_559__ macro.\n\n  Update: GCC does not per se set __STDC_IEC_559__, so also test for\n  __GCC_IEC_559 >= 1.","distinct":false,"url":"https://api.github.com/repos/opensourcerouting/quagga/commits/e226f00e96d5b3e03195d7f1c1d476453235916b"},{"sha":"65a0fe683845bbdd6b8f5299900572551e2a174f","author":{"name":"Paul Jakma","email":"54f65660711c038b3360dd7288dc12347ae58ba3@opensourcerouting.org"},"message":"lib: consolidate ntohf/htonf from ospfd/isisd TE to lib/network\n\n* lib/network.{c,h}: Consolidate the ntohf/htonf functions used in ospfd\n  TE to here, using the value-passing variant that fits with existing\n  ntoh/hton functions.\n* ospfd/ospf_opaque.c: Remove its variants.\n* ospfd/ospf_te.c: Update to use the consolidated, by-value variant.","distinct":false,"url":"https://api.github.com/repos/opensourcerouting/quagga/commits/65a0fe683845bbdd6b8f5299900572551e2a174f"},{"sha":"a80b9382b917b8b6ecd52b0635bde4a0b67d4721","author":{"name":"Olivier Dugeon","email":"69c8658fdd28ffd243b2886d83fa28cfbacee33e@orange.com"},"message":"lib: Add new if_link_params structure\n\nThis new structure is the basis to get new link parameters for\nTraffic Engineering from Zebra/interface layer to OSPFD and ISISD\nfor the support of Traffic Engineering\n\n* lib/if.[c,h]: link parameters struture and get/set functions\n* lib/command.[c,h]: creation of a new link-node\n* lib/zclient.[c,h]: modification to the ZBUS message to convey the\n  link parameters structure\n* lib/zebra.h: New ZBUS message\n* lib/memtypes.c: Add new memory type for Traffic Engineering support\n\nSigned-off-by: Olivier Dugeon <olivier.dugeon@orange.com>","distinct":false,"url":"https://api.github.com/repos/opensourcerouting/quagga/commits/a80b9382b917b8b6ecd52b0635bde4a0b67d4721"},{"sha":"74310f925d3257c37ca08f037df38240a9eb8696","author":{"name":"Olivier Dugeon","email":"69c8658fdd28ffd243b2886d83fa28cfbacee33e@orange.com"},"message":"zebra: Add link parameters support to Zebra\n\n* zebra/interface.c:\n   - Add new link-params CLI commands\n   - Add new functions to set/get link parameters for interface\n* zebra/redistribute.[c,h]: Add new function to propagate link parameters\n  to routing daemon (essentially OSPFD and ISISD) for Traffic Engineering.\n* zebra/redistribute_null.c: Add new function\n  zebra_interface_parameters_update()\n* zebra/zserv.[c,h]: Add new functions to send link parameters\n\nSigned-off-by: Olivier Dugeon <olivier.dugeon@orange.com>","distinct":false,"url":"https://api.github.com/repos/opensourcerouting/quagga/commits/74310f925d3257c37ca08f037df38240a9eb8696"},{"sha":"f8c5989f92661b9dde689c522a5eef180a27fed3","author":{"name":"Olivier Dugeon","email":"69c8658fdd28ffd243b2886d83fa28cfbacee33e@orange.com"},"message":"vtysh: Add support of new link-params CLI\n\nIn vtysh_config.c/vtysh_config_parse_line(), it is not possible to continue\nto use the ordered version for adding line i.e. config_add_line_uniq() to print\nInterface CLI commands as it completely break the new LINK_PARAMS_NODE.\n\nSigned-off-by: Olivier Dugeon <olivier.dugeon@orange.com>","distinct":false,"url":"https://api.github.com/repos/opensourcerouting/quagga/commits/f8c5989f92661b9dde689c522a5eef180a27fed3"},{"sha":"5947e0582a4927ea0a5d579299e451c838e6b630","author":{"name":"Olivier Dugeon","email":"69c8658fdd28ffd243b2886d83fa28cfbacee33e@orange.com"},"message":"ospfd: Update Traffic Engineering support\n\nThese patches update original code to RFC3630 (OSPF-TE) and add support of\nRFC5392 (Inter-AS v2) & RFC7471 (TE metric extensions) and partial support\nof RFC6827 (ASON - GMPLS).\n\n* ospfd/ospf_dump.[c,h]: Add new dump functions for Traffic Engineering\n* ospfd/ospf_opaque.[c,h]: Add new TLV code points for RFC5392\n* ospfd/ospf_packet.c: Update checking of OSPF_OPTION\n* ospfd/ospf_vty.[c,h]: Update ospf_str2area_id\n* ospfd/ospf_zebra.c: Add new function ospf_interface_link_params() to get\n  Link Parameters information from the interface to populate Traffic\n  Engineering metrics\n* ospfd/ospfd.[c,h]: Update OSPF_OPTION flags (T -> MT and new DN)\n* ospfd/ospf_te.[c,h]: Major modifications to update the code to new\n  link parameters structure and new RFCs\n\nSigned-off-by: Olivier Dugeon <olivier.dugeon@orange.com>","distinct":false,"url":"https://api.github.com/repos/opensourcerouting/quagga/commits/5947e0582a4927ea0a5d579299e451c838e6b630"},{"sha":"7740344dfb824f4444278a87fead5b2037e89027","author":{"name":"Olivier Dugeon","email":"69c8658fdd28ffd243b2886d83fa28cfbacee33e@orange.com"},"message":"isisd: Add support of Traffic Engineering\n\nThese patches is an implementation of RFC5305 that enable the\nsupport of Traffic Engineering in IS-IS\n\n* isisd/Makefile.am: Add new files isis_te.c and isis_te.h\n\n* isisd/isis_circuit.[c,h]: Add new mpls_te_circuit structure to\n  isis_circuit structure to handle new Traffic Engineering TLVs\n* isisd/isis_lsp.c: Update LSP handler to mux/demux Traffic Engineering TLVs\n* isisd/isis_main.c: Add initialisation of ISIS TE\n* isisd/isis_pdu.c: Update function process_p2p_hello() to retrieve remote\n  IP address to populate Traffic Engineering TLV.\n* isisd/isis_te.[c,]: Implementation of RFC5305\n* isisd/isis_tlv.[c,h]: Update TLV definition and function to handle\n  Traffic Engineering ones\n* isisd/isis_zebra.c: Add new function isis_zebra_link_params() to retrieve\n  the link parameters of interfaces from ZBus to populate the Traffic\n  Engineering TLVs\n* isisd/isisd.[c,h]: Add Traffic Engineering support with new debug command\n\nSigned-off-by: Olivier Dugeon <olivier.dugeon@orange.com>","distinct":false,"url":"https://api.github.com/repos/opensourcerouting/quagga/commits/7740344dfb824f4444278a87fead5b2037e89027"},{"sha":"e72b3aeb9a378e8ebe5286eb462f29fe6a87237c","author":{"name":"Olivier Dugeon","email":"69c8658fdd28ffd243b2886d83fa28cfbacee33e@orange.com"},"message":"doc: Update documentation for Traffic Engineering\n\nSigned-off-by: Olivier Dugeon <olivier.dugeon@orange.com>","distinct":false,"url":"https://api.github.com/repos/opensourcerouting/quagga/commits/e72b3aeb9a378e8ebe5286eb462f29fe6a87237c"}]}
{"actor":{"display_login":"mwinter-osr"}}