profile
viewpoint
Job Snijders job @nttgin @OpenBSD @NLNOG Amsterdam, Netherlands http://sobornost.net/~job/ Routing security & BGP expert at NTT, vice president PeeringDB, art director at OpenBSD

job/aggregate6 90

IPv4 and IPv6 prefix aggregation tool

job/aclhound 19

ACLHound

job/cdn.openbsd.org-ansible 14

Ansible playbooks to manage cdn.openbsd.org

job/bgp-ansible 8

Ansible repository for AS 15562 - my personal BGP speakers

hash-networker/wisdom 5

Wisdom from #networker

job/console.rpki-client.org 3

script to generate rpki console website

job/ansible-role-firewall 1

A role to manage iptables rules which doesn't suck.

job/birdseye 1

Birdseye Bird Looking Glass

issue openedirrdnet/irrd

Prevent unnecesary auth hash filtering in WhoisQueryResponse

WhoisQueryResponse runs the auth hash filter on all outputs, but this is only needed when auth hashes may be returned, i.e. not for !a. This has a performance impact of around 10%.

created time in 26 minutes

issue openedirrdnet/irrd

GraphQL queries are likely not filtering auth hashes

GraphQL queries should do auth hash filtering on both objectText and the auth attributes - they likely are not doing that now.

created time in 27 minutes

issue openedirrdnet/irrd

Make RPKI max-length available in DB and GraphQL query

The RPKI importer currently only saves the max-length in the pseudo-IRR object, in an attribute. For easier query support, it should also add it to parsed_data, to allow it to be included in the GraphQL interface, which should also be extended for this.

created time in an hour

push eventpmacct/pmacct

Paolo Lucente

commit sha c6c094aed664d381cd988d911044e202f4a91640

* nfacctd: adding support for MPLS VPN RD via IE #90 (7)

view details

Paolo Lucente

commit sha 15b12141bc15943030408a82dd0115461d6df51b

* Updated docs.

view details

push time in 2 hours

push eventirrdnet/irrd

Sasha Romijn

commit sha a13ea986cadd304f65fce69e7322b075e1cc693a

fix sa lag

view details

push time in 6 hours

create barnchirrdnet/irrd

branch : pypika-experiment

created branch time in 6 hours

issue commentytti/oxidized

APC_AOS Status Failed

We have the same issue. It would be good to get some traction on a fix.

fatpacket

comment created time in 10 hours

push eventNLNOG/ring-ansible

NLNOG Ring Admins - ring-admins@nlnog.net

commit sha 5f68fb139980f2ee17aad0e637194bdb4edeeb81

dbmaster commit by ringforger

view details

push time in 14 hours

push eventNLNOG/ring-ansible

NLNOG Ring Admins - ring-admins@nlnog.net

commit sha 4b1835ad14a55c56a9018cb1d769e81aa74458fc

dbmaster commit by ringforger

view details

push time in 14 hours

push eventNLNOG/ring-ansible

NLNOG Ring Admins - ring-admins@nlnog.net

commit sha 6d6527b00789cdc5c729deab6800dcf22388d693

dbmaster commit by ringforger

view details

push time in 18 hours

issue commentpeeringdb/peeringdb

IXF-Importer: reorder of email content

he email is not necessarily internal. Once #875 is implemented, the Importer generated Deskpro tickets may CC: external contacts when able. The "https://peeringdb.com//cp/peeringdb_server/ixfmemberdata/" URL is only present for use by the @peeringdb/ac, as a convenient link should an admin decide to make the suggested change. Users can not use it. Are you saying you don't want it to be included anymore?

Where did I say that? My remarks are only about the IX-F JSON URL.

This ticket was not originally about the IX-F JSON URL, it is about the internal /ixfmemberdata/ URL. Thus when you wrote:

From an @peeringdb/ac pov you could even drop this line.

I took that to be about the line indicated in the first message of this issue:

You may review and manually accept the IX-F data at https://peeringdb.com//cp/peeringdb_server/ixfmemberdata/nnnn/change/

Upon closer reading I see:

So, if you move the text to the 2nd last line and obey visibility we could even leave it in.

and I suppose you mean the IX-F JSON URL line in that context. It would be good to be explicit about the lines you are talking about.

Also, to be very clear, neither of these lines are present in the non-Deskpro emails the Importer sends out. It is not the intent of this issue to add lines to the non-Deskpro emails the Importer sends out. Rather these lines are only in the Deskpro tickets which are created, and which because of #875, will soon CC: networks/IXes upon creation, in addition to being seen by PeeringDB admins.

Thus I am guessing what you mean to propose is:

Summary:

For Deskpro ticket creation (and resulting emails), rearrange lines and factor in "IX-F Member Export URL Visibility" such that when that setting is public:

- IX-F Data: [URL]
- PeeringDB Admins Only: http://peeringdb.com/cp/peeringdb_server/ixfmemberdata/nnn/change/

When not public:

- IX-F Data: Not public
- PeeringDB Admins Only: http://peeringdb.com/cp/peeringdb_server/ixfmemberdata/nnn/change/
arnoldnipper

comment created time in a day

issue commentpeeringdb/peeringdb

IXF-Importer: reorder of email content

he email is not necessarily internal. Once #875 is implemented, the Importer generated Deskpro tickets may CC: external contacts when able.

The "https://peeringdb.com//cp/peeringdb_server/ixfmemberdata/" URL is only present for use by the @peeringdb/ac, as a convenient link should an admin decide to make the suggested change. Users can not use it. Are you saying you don't want it to be included anymore?

Where did I say that? My remarks are only about the IX-F JSON URL.

arnoldnipper

comment created time in a day

issue commentpeeringdb/peeringdb

IXF-Importer: reorder of email content

I deleted my previous comment after re-remembering these are two different things. @arnoldnipper, this is different than #875 (comment) which references the the IX-F JSON URL. Rather, here the reference is to an internal PeeringDB URL, not the IX-F JSON URL, thus I believe @egfrank's Summary is correct.

I know that these are two different things. Though this email is internal, @peeringdb/ac quotes it to send a message to the network. From an @peeringdb/ac pov you could even drop this line. The reason I want to have it at the bottom is to be able to easily remove it when quoting. So, if you move the text to the 2nd last line and obey visibility we could even leave it in.

The email is not necessarily internal. Once #875 is implemented, the Importer generated Deskpro tickets may CC: external contacts when able.

The "https://peeringdb.com//cp/peeringdb_server/ixfmemberdata/" URL is only present for use by the @peeringdb/ac, as a convenient link should an admin decide to make the suggested change. Users can not use it. Are you saying you don't want it to be included anymore?

arnoldnipper

comment created time in a day

issue commentpeeringdb/peeringdb

IXF-Importer: reorder of email content

I deleted my previous comment after re-remembering these are two different things. @arnoldnipper, this is different than #875 (comment) which references the the IX-F JSON URL. Rather, here the reference is to an internal PeeringDB URL, not the IX-F JSON URL, thus I believe @egfrank's Summary is correct.

I know that these are two different things. Though this email is internal, @peeringdb/ac quotes it to send a message to the network. From an @peeringdb/ac pov you could even drop this line. The reason I want to have it at the bottom is to be able to easily remove it when quoting.

arnoldnipper

comment created time in a day

issue commentpeeringdb/peeringdb

IXF-Importer: reorder of email content

Only publish the URL if ixf_ixp_member_list_url_visible = "Public", @egfrank

I deleted my previous comment after re-remembering these are two different things. @arnoldnipper, this is different than https://github.com/peeringdb/peeringdb/issues/875#issuecomment-731408103 which references the the IX-F JSON URL. Rather, here the reference is to an internal PeeringDB URL, not the IX-F JSON URL, thus I believe @egfrank's Summary is correct.

arnoldnipper

comment created time in a day

issue commentpeeringdb/peeringdb

IXF-Importer: reorder of email content

Only publish the URL if ixf_ixp_member_list_url_visible = "Public", @egfrank

@arnoldnipper, great point. @egfrank, copying from https://github.com/peeringdb/peeringdb/issues/875#issuecomment-731408103 :

Additional aspect to be included: Based on "IX-F Member Export URL Visibility" the Deskpro ticket created should only include the IX-F URL if visibility is set to "Public", since the email may go to parties other than the IX. When public:

- IX-F Data: [URL]

When not public:

- IX-F Data: Not public
arnoldnipper

comment created time in a day

issue commentpeeringdb/peeringdb

Add 'recommended_routeserver_asn' to ix object

UI would link to /net/$xxx, but display AS, and IP addresses. Display the IP since people use PDB's website to configure sessions by hand.

Please provide a mockup for the GUI and how it does look like in the API, @grizz

job

comment created time in a day

issue commentpeeringdb/peeringdb

IXF-Importer: reorder of email content

Only publish the URL if ixf_ixp_member_list_url_visible = "Public", @egfrank

arnoldnipper

comment created time in a day

push eventpmacct/pmacct

Paolo Lucente

commit sha aa7bffefb27982bfff827ba22d5230c92cfe8885

* nfacctd: adding support for MPLS VPN RD via IE #90 (6)

view details

push time in a day

push eventpeeringdb/translations

translate.peeringdb.com

commit sha 9c2efb0a6371f698f077d4b6e19b906385c76ef2

new translations (server:d7680a3 django:d249c9a django-oauth-toolkit:5cb5398)

view details

push time in a day

issue commentpeeringdb/peeringdb-py

Automatically update docs site on code release

@peeringdb/pc need another vote please, this might be a bug?

grizz

comment created time in a day

issue commentpeeringdb/peeringdb

Allow arbitray decimal places as input for longitude and latitude. Systems rounds to six

** Summary **

  • Allow arbitrary length of inputs to lat+long
  • Round to 6 digits (including google maps output)
arnoldnipper

comment created time in a day

issue commentpeeringdb/peeringdb

IXF-Importer: reorder of email content

** Summary **

  • Have link at the bottom of IXF emails that states
PeeringDB Admins Only: http://localhost/cp/peeringdb_server/ixfmemberdata/nnn/change/
arnoldnipper

comment created time in a day

issue commentpeeringdb/peeringdb

Add "Last login" to https://peeringdb.com/cp/peeringdb_server/user/

Summary

  • Add "last login" as a column in User list view in the admin portal (/cp/peeringdb_server/user/)
arnoldnipper

comment created time in a day

issue commentpeeringdb/peeringdb

wrap correctly on mobile

Summary

  • On mobile fac view, make ASN column wide enough to show 7 digits without wrapping
job

comment created time in a day

issue commentpeeringdb/peeringdb

Add 'recommended_routeserver_asn' to ix object

UI would link to /net/$xxx, but display AS, and IP addresses. Display the IP since people use PDB's website to configure sessions by hand.

job

comment created time in a day

issue commentpeeringdb/peeringdb

IX-F Importer: Add information to command error output to indicate which exchange and ix-f feed caused each error

Summary

  • Add exchange information (name, id, ix-f url) to IX-F command error output messages
vegu

comment created time in a day

push eventpeeringdb/peeringdb

Chris Caputo

commit sha d7680a3fb98b1bba714f56bb127e4ab3982f1c5f

Network IX-F Postmortem: "Preview"->"Postmortem" (#894)

view details

push time in a day

issue commentytti/oxidized

getting "version not found" error, when trying to look old version config file

seems like appending a default group is a workaround for this issue. Would be nice to have this bug fixed.

from88

comment created time in a day

more