-
Notifications
You must be signed in to change notification settings - Fork 11
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
Topology addHost is broken. #11
Comments
The syntax should really be underscore_fields for JSON, not CamelCase... That change was to make tests pass, and indeed it should be correct... JSON should really be underscore... So let's see what it broke - I mean which request was calling the addHost? |
You see, in form fields i'm using CamelCase, it's mostly recognised but response comes in underscore_case. I don't really care which one to use, but your tools are expecting CamelCase most of the time. I'm calling addHost in this script https://github.com/romana/romana/blob/test/core-pr/POC/ansible/poc-topology.sh |
Which revision is it? |
This PR is against feature/gg/external_id
Implement dynamic romana segment creation via CNI plugin
Implement dynamic romana segment creation via CNI plugin
This commit #ce9d91e1e85a3df5af93f7eaeb6fb57b2b5b2c07 introduces change that brakes syntax for registering new host in topology.
What is done: Adding a host using following syntax
Expecting result: Host created with all fields recognised
Actual result: Host created with romana_ip field empty
Additional information: since #ce9d91e1e85 following syntax works, but it mixes different spelling conventions together.
The text was updated successfully, but these errors were encountered: