Skip to content

ovs: ability to set other-config on bridge #61

Description

@breakintheweb

I see that we can only set openflow protocols as bridge options.

// Bridge sets configuration for a bridge using the values from a BridgeOptions
// struct.
func (v *VSwitchSetService) Bridge(bridge string, options BridgeOptions) error {
        // Prepend command line arguments before expanding options slice
        // and appending it
        args := []string{"set", "bridge", bridge}
        args = append(args, options.slice()...)
        log.Println(args)
        _, err := v.v.exec(args...)
        return err
}

Would it be possible to add another method to allow for setting something like "other-config:forward-bpdu=true", or possibly extend the BridgeOptions struct to allow 'other-options'

// BridgeOther allows for setting of non openflow BridgeOptions
func (v *VSwitchSetService) BridgeOther(bridge string, options string) error {
        // Prepend command line arguments before expanding options string
        // and appending it
        args := []string{"set", "bridge", bridge}
        args = append(args, options)
        _, err := v.v.exec(args...)
        return err
}

Thanks!

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Fields

    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions