)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":14525,"name":"Vasyl Saienko","email":"vsaienko@mirantis.com","username":"vsaienko"},"change_message_id":"32d78161e80ab8af89bb5ffdaa39177528d5ccb3","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Julian Edwards \u003cbigjools@gmail.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2017-03-17 15:20:02 +1000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Make running_in_container work with LXD"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id: I35e807c26f0b1fbba83ddbe04cfb4901a7a95cbe"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"1a1ced50_17658d44","line":7,"range":{"start_line":7,"start_character":36,"end_line":7,"end_character":39},"updated":"2017-03-17 09:19:35.000000000","message":"should detect not only LXD but other container apps like docker /openvz","commit_id":"397d93aadb869f82d774f3e2f23187fea728bcba"},{"author":{"_account_id":14525,"name":"Vasyl Saienko","email":"vsaienko@mirantis.com","username":"vsaienko"},"change_message_id":"baf4ba44c8a49d47bae0b32a3aac478dea05de49","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Sean Dague \u003csean@dague.net\u003e"},{"line_number":5,"context_line":"CommitDate: 2017-03-17 14:38:52 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Make running_in_container work with LXD"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id: I35e807c26f0b1fbba83ddbe04cfb4901a7a95cbe"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"1a1ced50_7c821902","line":7,"range":{"start_line":7,"start_character":36,"end_line":7,"end_character":39},"updated":"2017-03-17 14:42:09.000000000","message":"nit: not only with LXD","commit_id":"88d341a02c01b9a5f604f853438e667b115e44fe"}],"functions":[{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"04a959c4e28d3c7568e56547aea3945e54a04912","unresolved":false,"context_lines":[{"line_number":670,"context_line":"        return 0"},{"line_number":671,"context_line":"    fi"},{"line_number":672,"context_line":""},{"line_number":673,"context_line":"    # LXD containers are a bit different, need to check the environment."},{"line_number":674,"context_line":"    if sudo grep -q lxc /proc/1/environ; then"},{"line_number":675,"context_line":"        return 0"},{"line_number":676,"context_line":"    fi"}],"source_content_type":"application/x-shellscript","patch_set":1,"id":"3a1ff146_27d78924","line":673,"updated":"2017-03-15 23:57:01.000000000","message":"I remember seeing this go by and having no idea what was quite right and reading a few stackoverflow things and it seemed ok. \n\nso maybe this wants container\u003dlxc per [1]?\n\nbut also, after some more digging, maybe the best way is \"systemd-detect-virt --container\"?  [2] suggets that is the \"one true way\" and if grub uses it, we should be good ...\n\n\n\n[1] http://stackoverflow.com/questions/20010199/determining-if-a-process-runs-inside-lxc-docker/20010626#20010626\n[2] https://bugs.debian.org/cgi-bin/bugreport.cgi?bug\u003d813007","commit_id":"5c3542e0de19aca006ab0f17c148009f0706d9aa"},{"author":{"_account_id":14525,"name":"Vasyl Saienko","email":"vsaienko@mirantis.com","username":"vsaienko"},"change_message_id":"6463fe26ff7ee7b92d219f90b72c32e94dfdcaad","unresolved":false,"context_lines":[{"line_number":665,"context_line":""},{"line_number":666,"context_line":""},{"line_number":667,"context_line":"# running_in_container - Returns true otherwise false"},{"line_number":668,"context_line":"function running_in_container {"},{"line_number":669,"context_line":"    if [ $(systemd-detect-virt --container) \u003d \u0027none\u0027 ]; then"},{"line_number":670,"context_line":"        return 0"},{"line_number":671,"context_line":"    fi"}],"source_content_type":"application/x-shellscript","patch_set":2,"id":"1a1ced50_617363b2","line":668,"updated":"2017-03-17 08:05:55.000000000","message":"This function is used only for ignoring loading kernel modules when we run in container. Instead of updating this to support different distros/container systems. Why you can\u0027t check if modules is already loaded and do not try to load it second time?","commit_id":"397d93aadb869f82d774f3e2f23187fea728bcba"}]}
