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

peeringmacrov6 cannot be updated via the UI #212

Closed
apolyr opened this issue Jun 16, 2015 · 1 comment
Closed

peeringmacrov6 cannot be updated via the UI #212

apolyr opened this issue Jun 16, 2015 · 1 comment

Comments

@apolyr
Copy link

apolyr commented Jun 16, 2015

We use IXPmanger in order to produce our route server configuration. However, the v6 config for one of our customers was incorrect. After digging around, I see that in the database there are two different fields for AS-SETs:

peeringmacro varchar(255) COLLATE utf8_unicode_ci DEFAULT NULL,
peeringmacrov6 varchar(255) COLLATE utf8_unicode_ci DEFAULT NULL,

Indeed the customer has different values in these two fields:

INSERT INTO cust VALUES (1,1,1,1,NULL,'Greek Internet Exchange',1,'grix','GR-IX',199399,300,'XX','XX','XX','XX','XX','12x5','http://www.gr-ix.gr','cocolala','AS-GRIX','open','http://www.gr-ix.gr','2014-03-13',NULL,1,1,NULL,'2015-06-11',1,'superuser','2014-03-13','NO',0)

The second value seems to be a leftover; both fields should have the same value.

The peeringmacrov6 field cannot be updated through the UI (we run IXP Manager V3.7.2). The "edit customer details" button opens a page with a single "Peering Macro" field, which only updates the peeringmacro field of the cust table.

@barryo
Copy link
Member

barryo commented Jun 29, 2015

peeringmacrov6 is not a leftover but a quick hack for a IXP Manager user who had a member with different v4 and v6 macros.

The following commit adds the field to the UI.

Note that the IPv4 peering macro is used when there is no v6 macro set. To force no macro for IPv6, set it as AS-NULL. This caveat is including in the frontend.

@barryo barryo closed this as completed in d059073 Jun 29, 2015
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

No branches or pull requests

2 participants