編集履歴

質問編集履歴

脱字修正

2020/04/25 04:40

投稿

Yuta_for

スコア21

test CHANGED Viewed

File without changes

test CHANGED Viewed

@@ -362,6 +362,8 @@
+2020/4/25(土)追記
 頂いた回答を基に修正したコードです。
 ```Python

回答後の修正

2020/04/25 04:40

投稿

Yuta_for

スコア21

test CHANGED Viewed

File without changes

test CHANGED Viewed

@@ -34,112 +34,408 @@
+import sys
+import os.path
+import subprocess
+import re
+from paramiko import SSHConfig
+from io import StringIO
+from optparse import OptionParser
+from collections import defaultdict
+try:
+    import json
+except:
+    import simplejson as json
+_group = 'vagrant'  # a default group
-# Copyright (C) 2013  Mark Mandel <mark@compoundtheory.com>
+_ssh_to_ansible = [('user', 'ansible_ssh_user'),
+                   ('hostname', 'ansible_ssh_host'),
+                   ('identityfile', 'ansible_ssh_private_key_file'),
+                   ('port', 'ansible_ssh_port')]
+# Options
+# ------------------------------
-#               2015  Igor Khomyakov <homyakov@gmail.com>
+parser = OptionParser(usage="%prog [options] --list | --host <machine>")
+parser.add_option('--list', default=False, dest="list", action="store_true",
+                  help="Produce a JSON consumable grouping of Vagrant servers for Ansible")
+parser.add_option('--host', default=None, dest="host",
+                  help="Generate additional host specific details for given host for Ansible")
+(options, args) = parser.parse_args()
 #
-# This program is free software: you can redistribute it and/or modify
-# it under the terms of the GNU General Public License as published by
-# the Free Software Foundation, either version 3 of the License, or
-# (at your option) any later version.
+# helper functions
 #
+# get all the ssh configs for all boxes in an array of dictionaries.
+def get_ssh_config():
+    return dict((k, get_a_ssh_config(k)) for k in list_running_boxes())
+# list all the running boxes
+def list_running_boxes():
+    output = subprocess.check_output(["vagrant", "status"]).split('\n')
+    boxes = []
+    for line in output:
+        matcher = re.search("([^\s]+)[\s]+running (.+", line)
+        if matcher:
+            boxes.append(matcher.group(1))
+    return boxes
+# get the ssh config for a single box
+def get_a_ssh_config(box_name):
+    """Gives back a map of all the machine's ssh configurations"""
+    output = subprocess.check_output(["vagrant", "ssh-config", box_name])
+    config = SSHConfig()
+    config.parse(StringIO(output))
+    host_config = config.lookup(box_name)
+    # man 5 ssh_config:
-# This program is distributed in the hope that it will be useful,
+    # > It is possible to have multiple identity files ...
-# but WITHOUT ANY WARRANTY; without even the implied warranty of
+    # > all these identities will be tried in sequence.
+    for id in host_config['identityfile']:
+        if os.path.isfile(id):
+            host_config['identityfile'] = id
-# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+    return dict((v, host_config[k]) for k, v in _ssh_to_ansible)
+# List out servers that vagrant has running
+# ------------------------------
+if options.list:
+    ssh_config = get_ssh_config()
+    meta = defaultdict(dict)
+    for host in ssh_config:
+        meta['hostvars'][host] = ssh_config[host]
+    print((json.dumps({_group: list(ssh_config.keys()), '_meta': meta})))
+    sys.exit(0)
-# GNU General Public License for more details.
+# Get out the host details
+# ------------------------------
+elif options.host:
+    print((json.dumps(get_a_ssh_config(options.host))))
+    sys.exit(0)
+# Print out help
+# ------------------------------
+else:
+    parser.print_help()
+    sys.exit(0)
+実行結果
+yuta@yuta-PC:~/Desktop/work/ansible/ansible-tutorial$ ./vagrant.py --list
+Traceback (most recent call last):
+  File "./vagrant.py", line 108, in <module>
+    ssh_config = get_ssh_config()
+  File "./vagrant.py", line 70, in get_ssh_config
+    return dict((k, get_a_ssh_config(k)) for k in list_running_boxes())
+  File "./vagrant.py", line 75, in list_running_boxes
+    output = subprocess.check_output(["vagrant", "status"]).split('\n')
+TypeError: a bytes-like object is required, not 'str'
+```
+書籍の想定出力ですと以下のようになるみたいです。
+```Python
+{
+  "vagrant": [
+    "default"
+  ],
+  "_meta": {
+   "hostvars": {
+     "default": {
+       "ansible_ssh_host": "127.0.0.1",
+       "ansible_ssh_port": "2222",
+       "ansible_ssh_user": "vagrant",
+       "ansible_ssh_private_key_file": "Users/username/ansible-tutorial/.vagrant/machines/default/virtualbox/private_key"
+     }
+    }
+   }
+}
+```
+もともとPython2系で書かれていたみたいで、2to3ツールで３に変換した上で実行したのですが、エラーが発生しました。
+以前も[同じようなエラーが出た時に質問した](https://teratail.com/questions/253116)のですが、
+TypeError: a bytes-like object is required, not 'str' 左記のエラーから文字コードを指定してバイト型に変換する必要があるとのことで、
+output = subprocess.check_output([**b**"vagrant", **b**"status"]).split('\n')
+上記のようにバイト変換をしましたが、結果は変わりませんでした。
+前提条件となるVagrantの起動は完了しております。
+他にどうすればよいのか分からなくてわかる人がいましたらご教授お願いいたします。
+追記
+レシーバにdecode()を修正して実行しましたが、別エラーが発生しました。
+```Python
+# list all the running boxes
+def list_running_boxes():
+    output = subprocess.check_output(["vagrant", "status"]).decode().split('\n')
+    boxes = []
+    for line in output:
+        matcher = re.search("([^\s]+)[\s]+running (.+", line)
+        if matcher:
+            boxes.append(matcher.group(1))
+    return boxes
+実行結果
+yuta@yuta-PC:~/Desktop/work/ansible/ansible-tutorial$ ./vagrant.py --list
+Traceback (most recent call last):
+  File "./vagrant.py", line 108, in <module>
+    ssh_config = get_ssh_config()
+  File "./vagrant.py", line 70, in get_ssh_config
+    return dict((k, get_a_ssh_config(k)) for k in list_running_boxes())
+  File "./vagrant.py", line 70, in <genexpr>
+    return dict((k, get_a_ssh_config(k)) for k in list_running_boxes())
+  File "./vagrant.py", line 93, in get_a_ssh_config
+    config.parse(StringIO(output))
+TypeError: initial_value must be str or None, not bytes
+```
+頂いた回答を基に修正したコードです。
+```Python
+#!/usr/bin/env python
+import sys
+import os.path
+import subprocess
+import re
+from paramiko import SSHConfig
+from io import BytesIO
+from optparse import OptionParser
+from collections import defaultdict
+try:
+    import json
+except:
+    import simplejson as json
+_group = 'vagrant'  # a default group
+_ssh_to_ansible = [('user', 'ansible_ssh_user'),
+                   ('hostname', 'ansible_ssh_host'),
+                   ('identityfile', 'ansible_ssh_private_key_file'),
+                   ('port', 'ansible_ssh_port')]
+# Options
+# ------------------------------
+parser = OptionParser(usage="%prog [options] --list | --host <machine>")
+parser.add_option('--list', default=False, dest="list", action="store_true",
+                  help="Produce a JSON consumable grouping of Vagrant servers for Ansible")
+parser.add_option('--host', default=None, dest="host",
+                  help="Generate additional host specific details for given host for Ansible")
+(options, args) = parser.parse_args()
 #
-# You should have received a copy of the GNU General Public License
+# helper functions
-# along with this program.  If not, see <http://www.gnu.org/licenses/>.
 #
-# Thanks to the spacewalk.py inventory script for giving me the basic structure
-# of this.
-#
-import sys
-import os.path
-import subprocess
-import re
-from paramiko import SSHConfig
-from io import StringIO
-from optparse import OptionParser
-from collections import defaultdict
-try:
-    import json
-except:
-    import simplejson as json
-_group = 'vagrant'  # a default group
-_ssh_to_ansible = [('user', 'ansible_ssh_user'),
-                   ('hostname', 'ansible_ssh_host'),
-                   ('identityfile', 'ansible_ssh_private_key_file'),
-                   ('port', 'ansible_ssh_port')]
-# Options
-# ------------------------------
-parser = OptionParser(usage="%prog [options] --list | --host <machine>")
-parser.add_option('--list', default=False, dest="list", action="store_true",
-                  help="Produce a JSON consumable grouping of Vagrant servers for Ansible")
-parser.add_option('--host', default=None, dest="host",
-                  help="Generate additional host specific details for given host for Ansible")
-(options, args) = parser.parse_args()
-#
-# helper functions
-#
@@ -158,7 +454,7 @@
 def list_running_boxes():
-    output = subprocess.check_output(["vagrant", "status"]).split('\n')
+    output = subprocess.check_output([b"vagrant", b"status"]).split(b'\n')  #bを追加してbyte変換
@@ -168,7 +464,7 @@
     for line in output:
-        matcher = re.search("([^\s]+)[\s]+running (.+", line)
+        matcher = re.search(b"([^\s]+)[\s]+running (.+", line)        #bを追加してbyte変換
         if matcher:
@@ -194,7 +490,7 @@
     config = SSHConfig()
-    config.parse(StringIO(output))
+    config.parse(BytesIO(output))
     host_config = config.lookup(box_name)
@@ -266,138 +562,42 @@
-実行結果
+実行
-yuta@yuta-PC:~/Desktop/work/ansible/ansible-tutorial$ ./vagrant.py --list
+python vagrant.py --list
 Traceback (most recent call last):
-  File "./vagrant.py", line 108, in <module>
+  File "vagrant.py", line 108, in <module>
     ssh_config = get_ssh_config()
-  File "./vagrant.py", line 70, in get_ssh_config
+  File "vagrant.py", line 70, in get_ssh_config
     return dict((k, get_a_ssh_config(k)) for k in list_running_boxes())
+  File "vagrant.py", line 70, in <genexpr>
+    return dict((k, get_a_ssh_config(k)) for k in list_running_boxes())
-  File "./vagrant.py", line 75, in list_running_boxes
+  File "vagrant.py", line 93, in get_a_ssh_config
+    config.parse(BytesIO(output))
+  File "/home/yuta/.pyenv/versions/3.8.2/lib/python3.8/site-packages/paramiko/config.py", line 136, in parse
-    output = subprocess.check_output(["vagrant", "status"]).split('\n')
+    if not line or line.startswith("#"):
-TypeError: a bytes-like object is required, not 'str'
+TypeError: startswith first arg must be bytes or a tuple of bytes, not str
 ```
-書籍の想定出力ですと以下のようになるみたいです。
-```Python
-{
-  "vagrant": [
-    "default"
-  ],
-  "_meta": {
-   "hostvars": {
-     "default": {
-       "ansible_ssh_host": "127.0.0.1",
-       "ansible_ssh_port": "2222",
-       "ansible_ssh_user": "vagrant",
-       "ansible_ssh_private_key_file": "Users/username/ansible-tutorial/.vagrant/machines/default/virtualbox/private_key"
-     }
-    }
-   }
-}
-```
-もともとPython2系で書かれていたみたいで、2to3ツールで３に変換した上で実行したのですが、エラーが発生しました。
-以前も[同じようなエラーが出た時に質問した](https://teratail.com/questions/253116)のですが、
+エラーの内容から最初の引数をバイトにしないといけないと出力されていますが、エラー先がpyenv以下のparamikoパッケージのコンフィグファイルであり、この部分を修正するということでしょうか？
-TypeError: a bytes-like object is required, not 'str' 左記のエラーから文字コードを指定してバイト型に変換する必要があるとのことで、
-output = subprocess.check_output([**b**"vagrant", **b**"status"]).split('\n')
-上記のようにバイト変換をしましたが、結果は変わりませんでした。
-前提条件となるVagrantの起動は完了しております。
-他にどうすればよいのか分からなくてわかる人がいましたらご教授お願いいたします。
+pip でインストールしたモジュールのコンフィグファイルを変更することに少し不安感を覚えるのですがこの考えは間違っていないのかご教授お願いいたします。
-追記
-レシーバにdecode()を修正して実行しましたが、別エラーが発生しました。
-```Python
-# list all the running boxes
-def list_running_boxes():
-    output = subprocess.check_output(["vagrant", "status"]).decode().split('\n')
-    boxes = []
-    for line in output:
-        matcher = re.search("([^\s]+)[\s]+running (.+", line)
-        if matcher:
-            boxes.append(matcher.group(1))
-    return boxes
-実行結果
-yuta@yuta-PC:~/Desktop/work/ansible/ansible-tutorial$ ./vagrant.py --list
-Traceback (most recent call last):
-  File "./vagrant.py", line 108, in <module>
-    ssh_config = get_ssh_config()
-  File "./vagrant.py", line 70, in get_ssh_config
-    return dict((k, get_a_ssh_config(k)) for k in list_running_boxes())
-  File "./vagrant.py", line 70, in <genexpr>
-    return dict((k, get_a_ssh_config(k)) for k in list_running_boxes())
-  File "./vagrant.py", line 93, in get_a_ssh_config
-    config.parse(StringIO(output))
-TypeError: initial_value must be str or None, not bytes
-```

追記分の追加

2020/04/25 04:39

投稿

Yuta_for

スコア21

test CHANGED Viewed

File without changes

test CHANGED Viewed

@@ -6,7 +6,7 @@
 ##エラーの詳細
-Ansibleの勉強のために以下の技術書に記載されているDynamic Inventoryスクリプトを起動しましたが型エラーでつまずいてしまいました。
+Ansibleの勉強のために以下の[技術書](https://www.amazon.co.jp/Ansible%E5%BE%B9%E5%BA%95%E5%85%A5%E9%96%80-%E3%82%AF%E3%83%A9%E3%82%A6%E3%83%89%E6%99%82%E4%BB%A3%E3%81%AE%E6%96%B0%E3%81%97%E3%81%84%E6%A7%8B%E6%88%90%E7%AE%A1%E7%90%86%E3%81%AE%E5%AE%9F%E7%8F%BE-%E5%BB%A3%E5%B7%9D-%E8%8B%B1%E5%AF%BF/dp/4798149942)に記載されているDynamic Inventoryスクリプトを起動しましたが型エラーでつまずいてしまいました。
 ```Python
@@ -341,3 +341,63 @@
 前提条件となるVagrantの起動は完了しております。
 他にどうすればよいのか分からなくてわかる人がいましたらご教授お願いいたします。
+追記
+レシーバにdecode()を修正して実行しましたが、別エラーが発生しました。
+```Python
+# list all the running boxes
+def list_running_boxes():
+    output = subprocess.check_output(["vagrant", "status"]).decode().split('\n')
+    boxes = []
+    for line in output:
+        matcher = re.search("([^\s]+)[\s]+running (.+", line)
+        if matcher:
+            boxes.append(matcher.group(1))
+    return boxes
+実行結果
+yuta@yuta-PC:~/Desktop/work/ansible/ansible-tutorial$ ./vagrant.py --list
+Traceback (most recent call last):
+  File "./vagrant.py", line 108, in <module>
+    ssh_config = get_ssh_config()
+  File "./vagrant.py", line 70, in get_ssh_config
+    return dict((k, get_a_ssh_config(k)) for k in list_running_boxes())
+  File "./vagrant.py", line 70, in <genexpr>
+    return dict((k, get_a_ssh_config(k)) for k in list_running_boxes())
+  File "./vagrant.py", line 93, in get_a_ssh_config
+    config.parse(StringIO(output))
+TypeError: initial_value must be str or None, not bytes
+```