Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update ixf_member_list_from_clients.py & Create rtt_getter.c #67

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

happyhater
Copy link
Contributor

Hello,

I created rtt_getter in C to works on OpenBSD (natively) or even on Linux.
Also there's an major edit for IX-F where I think are mandatory to use 'if_speed',

Thanks!
--z

RTT getter program for RTT-based actions for OpenBSD or Linux.
Adding 'if_speed' -- It's used on PeeringDB for showing link speed of a member.
@pierky
Copy link
Owner

pierky commented Mar 13, 2021

Hello @happyhater,

just a clarification on the Euro-IX JSON file. You say

Also there's an major edit for IX-F where I think are mandatory to use 'if_speed',

Where did you see this?

I've recently added a test case that's used to validate the output of IXFMemberListFromClientsCommand.build_json (so, the sub-command ixf-member-export): the test uses the Euro-IX API endpoint from https://ixpdb.euro-ix.net/en/validator/ to submit the JSON and verify that no errors are reported, and it actually reports no errors.

Also, looking at the 1.0 specification I can't see if_speed being mentioned as a mandatory/required attributes of any structure in the schema.

Could you clarify what makes you thinking that if_speed is a required field? Thanks

@happyhater
Copy link
Contributor Author

happyhater commented Mar 14, 2021

                            "if_speed": {
                              "description": "Speed of the interface or LAG in Mb, i.e. 10G = 10000",
                              "type": "integer"
                            },

That's true, this function are used only on PDB. #L262
In my opinion I think is useful for showing the port speed with Internet Exchange, most of them don't know what is their 'if_speed' allocated, or use a false setup.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants