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!

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions