Use OVSDB client in order to retrive port list

[shun159]

https://3.basecamp.com/3088280/buckets/46054/messages/148393356
の通り、OVSDBを使った実装を試しに書いてみました。テストはまだありません。

[houndci-bot]

Prefer single-quoted strings when you don't need string interpolation or special symbols.

[yasuhito]

ここで shun159/active_flow に依存してしまうと、依存先が trema/active_flow と phut の active_flow とこの active_flow で 3 つになってしまうので 笑、まずは phut のほうにクラスをコピーしてもらうのが良いと思います (ライセンスなど問題がなければ)

[shun159]

しょうちしました。

[shun159]

コピーしてきました。
7ee0730

[houndci-bot]

Prefer single-quoted strings when you don't need string interpolation or special symbols.

[yasuhito]

まずテストについては、vswitch#ports.feature がそのまま使えると思います。この最後のシナリオでは、ポート 2 番に仮想リンクをつなげたときの Vswitch#ports の出力チェックをしています。

Scenario: Vswitch#ports
  Given I type "link = Link.create('a', 'b')"
  And I type "vswitch.add_numbered_port 2, link.device('a')"
  When I type "vswitch.ports"
  And sleep 5
  Then the output should contain:
   """
   ["L0_a"]
   """

ここはたとえば [#<Port number: 2, device: "L0_a">] という出力が返ってくればカッコイイと思います。

[shun159]

テストを書きかきなおしてみました。
また add_numbered_portで実行するのは、"other_config:rstp-port-num=#{port_number}"では、
open vswitchでは、openflowポートのアサインができないので、"ofport_request=#{port_number}"で書きなおしています。

[shun159]

出力に変な改行が入ってしまい、本当は、

[#<OpenStruct name="vsw_0xabc", ofport=65534>, #<OpenStruct name="L0_a", ofport=2>]

と書きたいのですが…。

  Scenario: Vswitch#ports
    Given I type "link = Link.create('a', 'b')"
    And I type "vswitch.add_numbered_port 2, link.device('a')"
    When I type "vswitch.ports"
    And sleep 5
    Then the output should contain:
    """
    #<OpenStruct name="L0_a", ofport=2>
    ""

[yasuhito]

inspect の出力は、ActiveRecord 方式で アトリビュート名: 値 がよいです。
phut の OpenVswitch クラスも、そちらに合わせてありますです。
https://github.com/trema/phut/blob/develop/lib/phut/open_vswitch.rb#L91

あとは OpenStruct という名前も何だかわからないので、やはり Port がよいと思います。

[houndci-bot]

Missing frozen string literal comment.

[shun159]

Portクラスになるように、直してみました。
また、同名のbridgeやportが残っているままで、再度実行した時にbridgeやportの作成に失敗することを回避するために、いくつか実行しているコマンドに修正を加えました。

[yasuhito]

デバイス名とポート番号ではユーザが興味あるのはたいていポート番号のほうなので、

[#<Port number: 1, device: "L0_a">]

のほうが良いと思います。

[shun159]

しょうちしました。

[houndci-bot]

Missing frozen string literal comment.

[shun159]

修正しました。