ある日、Ansible 2.5.2 のバグに遭遇しました。 そのバグ修正に協力してみた話です。

1. IaC技術推進部 重点技術戦略室
横地 晃（@akira6592）
2018/05/18 AP Tech Fest
使う側から作る側へ 越境 するための第一歩

2. 使う側
はじめに
 先日 Ansible 2.5.2 のバグに遭遇しました。
 そのバグ修正に協力してみた話です。
 OSSに貢献してみたくても、ハードルの高さを
感じている人が、小さなことから始めてみよう
と思っていただけたら幸いです。
2
作る側
越境この辺のお話

3. 3
 バグ遭遇
 issue検索
 発生条件の調査
 issue上のやりとり
 検証記事の準備
 まとめ

4. 4
 バグ遭遇
 issue検索
 発生条件の調査
 issue上のやりとり
 検証記事の準備
 まとめ

5. バグ遭遇
5
ios_config モジュールにべき等性のバグが
 社内のプチ勉強会で、ルーターへのコンフィグ
投入をデモしていたら・・
 実行結果が意図せず毎回 *1になる
[vagrant@centos7 vagrant]$ ansible-playbook -i inventory ios_test.yml
PLAY [ios] ****************************************************
TASK [confit test] **********************************************
changed: [172.16.0.1]
PLAY RECAP **************************************************
172.16.0.1 : ok=1 changed=1 unreachable=0 failed=0
(*1) changed: 対象機器に実際に設定変更が発生した状態。Playbook実行2回目以降はchangedにならないのが正しい。
[vagrant@centos7 vagrant]$ ansible-playbook -i inventory ios_test.yml
PLAY [ios] ****************************************************
TASK [confit test] **********************************************
changed: [172.16.0.1]
PLAY RECAP **************************************************
172.16.0.1 : ok=1 changed=1 unreachable=0 failed=0
[vagrant@centos7 vagrant]$ ansible-playbook -i inventory ios_test.yml
PLAY [ios] ****************************************************
TASK [config test] **********************************************
changed: [172.16.0.1]
PLAY RECAP **************************************************
172.16.0.1 : ok=1 changed=1 unreachable=0 failed=0
＼ｶﾜｯﾀﾖ／

6. 6
 バグ遭遇
 issue検索
 発生条件の調査
 issue上のやりとり
 検証記事の準備
 まとめ

7. Issue検索
7
あったけど詳細な記載なし・未修正
https://github.com/ansible/ansible/issues/37550

8. 8
発生条件は自分で調べることに

9. 9
発生条件は自分で調べることに

10. 10
 バグ遭遇
 issue検索
 発生条件の調査
 issue上のやりとり
 検証記事の準備
 まとめ

11. 発生条件の調査 その1
11
毎回 changed になるコマンドとならないコマンドがある
毎回 changed になってしまうケース
snmp-server community apc RO
毎回 changed にならないケース（問題なし）
hostname rt999
access-list 1 permit 192.168.1.0 0.0.0.255
どういう条件
なんだろう・・
※Playbook内でコマンドを省略系で指定すると毎回 changed になるが、今回はそれとも異なる。

12. 12
そういえば、比較用に事前コンフィグを
「show run」 で取得する処理があって、
オプション次第で「show run all」
にもできるけど、何か関係あるかな。
ネットワークエンジニア的な勘
＋
Ansibleのコードを読んだ経験

13. 発生条件の調査 その2
13
「show run」 と 「show run all」 とで
コマンド表記が異なるコマンドがダメ
毎回 changed になってしまうケース
show run の結果 snmp-server community apc RO
show run all の結果 snmp-server community apcv1default RO
Catalyst 3750G-24T-E IOS 12.2(46)SE での検証
毎回 changed にならないケース（問題なし）
show run の結果 hostname rt999
show run all の結果 hostname rt999
同じ
異なる
なんかくっついて出る

14. 14
 バグ遭遇
 issue検索
 発生条件の調査
 issue上のやりとり
 検証記事の準備
 まとめ

15. issueへのコメント
15
「show run all と比較しちゃってるからですね」

16. その後
16
問題個所が特定、プルリクが作成される
https://github.com/ansible/ansible/pull/39475
※厳密には同じ原因の他のissueに対応するプルリク
1.ここが問題だね
2.直したよ
3.あっちのプルリクで解決

17. 17
あとはリリースを待つだけだ

18. 18
と思うじゃん？

19. さらに
19
類似の別のissueが追加されてた
https://github.com/ansible/ansible/pull/39724
前からそうでしたね。設計上の仕様なのでは。
（結局バグ扱い）

20. その後・・
20
修正者に動作確認を頼まれた
https://github.com/ansible/ansible/pull/39724
直したからキミ試してよ
あっハイ、うまくいきました。

21. 21
今度こそ・・・

22. 22
(アドリブ)

23. 23
本日 5/18
Ansible 2.5.3
無事リリース

24. 24
 バグ遭遇
 issue検索
 発生条件の調査
 issueへのコメント
 検証記事の準備
 まとめ

25. 予め本件の検証記事を準備
25
リリース前から情報を先取り
Ansible 2.5.3 リリース後、
最終検証後に公開予定

26. 予め本件の検証記事を準備
26
リリース前から情報を先取り
Ansible 2.5.3 リリース後、
最終検証後に公開予定
本日 5/18 15:00 頃
公開済み
・Ansible 2.5.3 で ios_config モジュールのコンフィグバックアップ時に defaults オプションを参照するようになった
http://tekunabe.hatenablog.jp/entry/2018/05/18/ansible_backup
・Ansible ios_config モジュールで意図せず chaneged になってしまうバグが 2.5.3 で修正された
http://tekunabe.hatenablog.jp/entry/2018/05/18/ansible_Idempotency_issu

27. 27
 バグ遭遇
 issue検索
 発生条件の調査
 issueへのコメント
 検証記事の準備
 まとめ

28. まとめ
28
コーディングだけでなく、issue をあげたり、
コメントすることでも OSS に貢献できる
使う側 作る側
越境
コメント
プル
リク
情報を早く掴むには、使う側から作る側へ
issue
越境するには階段が必要
自己肯定感
重要