{"action":"created","issue":{"url":"https://api.github.com/repos/ganeti/ganeti/issues/916","repository_url":"https://api.github.com/repos/ganeti/ganeti","labels_url":"https://api.github.com/repos/ganeti/ganeti/issues/916/labels{/name}","comments_url":"https://api.github.com/repos/ganeti/ganeti/issues/916/comments","events_url":"https://api.github.com/repos/ganeti/ganeti/issues/916/events","html_url":"https://github.com/ganeti/ganeti/issues/916","id":238300055,"number":916,"title":"Documentation for -H serial_console","user":{"login":"Ganeti-Issues-Migrator","id":29254221,"avatar_url":"https://avatars3.githubusercontent.com/u/29254221?v=3","gravatar_id":"","url":"https://api.github.com/users/Ganeti-Issues-Migrator","html_url":"https://github.com/Ganeti-Issues-Migrator","followers_url":"https://api.github.com/users/Ganeti-Issues-Migrator/followers","following_url":"https://api.github.com/users/Ganeti-Issues-Migrator/following{/other_user}","gists_url":"https://api.github.com/users/Ganeti-Issues-Migrator/gists{/gist_id}","starred_url":"https://api.github.com/users/Ganeti-Issues-Migrator/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Ganeti-Issues-Migrator/subscriptions","organizations_url":"https://api.github.com/users/Ganeti-Issues-Migrator/orgs","repos_url":"https://api.github.com/users/Ganeti-Issues-Migrator/repos","events_url":"https://api.github.com/users/Ganeti-Issues-Migrator/events{/privacy}","received_events_url":"https://api.github.com/users/Ganeti-Issues-Migrator/received_events","type":"User","site_admin":false},"labels":[{"id":631648911,"url":"https://api.github.com/repos/ganeti/ganeti/labels/imported_from_google_code","name":"imported_from_google_code","color":"ededed","default":false},{"id":632133180,"url":"https://api.github.com/repos/ganeti/ganeti/labels/Status:Fixed","name":"Status:Fixed","color":"ededed","default":false}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":3,"created_at":"2017-06-24T06:46:45Z","updated_at":"2017-06-24T23:04:44Z","closed_at":null,"body":"Originally reported of Google Code with ID 860.\n\n```\n[ganeti 2.10.4, wheezy-backports]\r\n\r\nIt's not obvious that the hypervisor setting serial_console does two distinct things:\r\n\r\n1. Enables emulation of a serial port (e.g. KVM: adds -serial .... option)\r\n2. Unconditionally appends \"console=ttyS0,<serial_speed>\" to the end of kernel_args, at least for KVM (Note: I cannot find this behaviour in code for Xen)\r\n\r\nThe *last* console=... setting on the kernel cmdline has special significance, because this is the one which is used for /dev/console.\r\n\r\nTherefore a consequence of point (2) is that it is impossible to configure something like\r\nkernel_args=\"console=ttyS0,9600 console=tty0\"\r\nso that the serial console is secondary but the VGA console is default.\r\n\r\nSuggestions:\r\n\r\n1. Clearer documentation in man gnt-instance. This behaviour could do with documenting under both \"serial_console\" and \"kernel_args\", so that you are aware that the actual kernel command line is not just what you set in kernel_args.\r\n\r\n2. Maybe the automatic appending of \"console=ttyS0,<serial_speed>\" should be suppressed if kernel_args already contains a \"console=\" string? (That would be a minor behaviour change)\r\n\r\nAnalysis:\r\nhttps://groups.google.com/forum/#!topic/ganeti/gz6D3jq0nTo\r\n\r\nKernel docs on the behaviour of console= kernel args:\r\nhttps://www.kernel.org/doc/Documentation/serial-console.txt\r\n\r\nCode reference:\r\n[lib/hypervisor/hv_kvm/__init__.py]\r\n        root_append.append(\"console=ttyS0,%s\" % serial_speed)\r\n\n```\nOriginally added on 2014-06-17 20:51:15 +0000 UTC."},"comment":{"url":"https://api.github.com/repos/ganeti/ganeti/issues/comments/310870928","html_url":"https://github.com/ganeti/ganeti/issues/916#issuecomment-310870928","issue_url":"https://api.github.com/repos/ganeti/ganeti/issues/916","id":310870928,"user":{"login":"Ganeti-Issues-Migrator","id":29254221,"avatar_url":"https://avatars3.githubusercontent.com/u/29254221?v=3","gravatar_id":"","url":"https://api.github.com/users/Ganeti-Issues-Migrator","html_url":"https://github.com/Ganeti-Issues-Migrator","followers_url":"https://api.github.com/users/Ganeti-Issues-Migrator/followers","following_url":"https://api.github.com/users/Ganeti-Issues-Migrator/following{/other_user}","gists_url":"https://api.github.com/users/Ganeti-Issues-Migrator/gists{/gist_id}","starred_url":"https://api.github.com/users/Ganeti-Issues-Migrator/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Ganeti-Issues-Migrator/subscriptions","organizations_url":"https://api.github.com/users/Ganeti-Issues-Migrator/orgs","repos_url":"https://api.github.com/users/Ganeti-Issues-Migrator/repos","events_url":"https://api.github.com/users/Ganeti-Issues-Migrator/events{/privacy}","received_events_url":"https://api.github.com/users/Ganeti-Issues-Migrator/received_events","type":"User","site_admin":false},"created_at":"2017-06-24T23:04:44Z","updated_at":"2017-06-24T23:04:44Z","body":"\n```\nDocumentation fixed with:\r\n\r\ncommit 14b9bda2af3acf4371a933d6da05db746b51d987\r\nAuthor: Lisa Velden <velden@google.com>\r\nDate:   Tue Jul 21 14:34:42 2015 +0200\r\n\r\n    Make documentation for -H serial_console more explicit\r\n    \r\n    Mention that apart from enabling the emulation of a serial port in KVM\r\n    \"console=ttyS0,<serial_speed>\" is appended to the end of kernel_args.\r\n    \r\n    Signed-off-by: Lisa Velden <velden@google.com>\r\n    Reviewed-by: Klaus Aehlig <aehlig@google.com>\n```\nOriginally added on 2015-10-21 09:07:44 +0000 UTC.\n\nRemoved Labels: Type-Documentation Priority-Low SmallTask\nChanged State: Fixed\nRemoved from Milestone: Release2.16"}}
{"actor":{"display_login":"Ganeti-Issues-Migrator"}}