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

/v2/flows returns flows with cookie_mask attribute #158

Open
Ktmi opened this issue Jul 21, 2023 · 1 comment
Open

/v2/flows returns flows with cookie_mask attribute #158

Ktmi opened this issue Jul 21, 2023 · 1 comment
Labels
bug Something isn't working

Comments

@Ktmi
Copy link

Ktmi commented Jul 21, 2023

When performing a GET operation on /v2/flows, the returned flows will contain the cookie_mask attribute set to 0. According to discussion with @viniarck this is not supposed to occur. Here's some example output from /v2/flows:

{
    "00:00:00:00:00:00:00:04": {
        "flows": [
            {
                "switch": "00:00:00:00:00:00:00:04",
                "table_id": 0,
                "match": {
                    "dl_vlan": 3799,
                    "dl_type": 35020
                },
                "priority": 50000,
                "idle_timeout": 0,
                "hard_timeout": 0,
                "cookie": 12321848580485677060,
                "id": "9f65bd3de40897e0e3396290a800881f",
                "stats": {
                    "byte_count": 9198,
                    "duration_sec": 655,
                    "duration_nsec": 831000000,
                    "packet_count": 219
                },
                "cookie_mask": 0,
                "instructions": [
                    {
                        "instruction_type": "apply_actions",
                        "actions": [
                            {
                                "port": 4294967293,
                                "action_type": "output"
                            }
                        ]
                    }
                ]
            },
            {
                "switch": "00:00:00:00:00:00:00:04",
                "table_id": 0,
                "match": {
                    "dl_src": "ee:ee:ee:ee:ee:03"
                },
                "priority": 50000,
                "idle_timeout": 0,
                "hard_timeout": 0,
                "cookie": 12393906174523604996,
                "id": "50f9877451949afdd4aaf9de121cdcff",
                "stats": {
                    "byte_count": 0,
                    "duration_sec": 655,
                    "duration_nsec": 677000000,
                    "packet_count": 0
                },
                "cookie_mask": 0,
                "instructions": [
                    {
                        "instruction_type": "apply_actions",
                        "actions": [
                            {
                                "port": 4294967293,
                                "action_type": "output"
                            }
                        ]
                    }
                ]
            }
        ]
    }
}
@Ktmi Ktmi added the bug Something isn't working label Jul 21, 2023
@viniarck
Copy link
Member

It could get omitted here, it won't cause issues with having the cookie_mask: 0, although it can lead to confusion indeed in a context where it's just showing/listing. The cookie_mask on of_core 0x04 Flow is still handy when converting to FlowMods.

Thanks for looking into it, and mapping this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants