Bug 1303635
Summary: | [ovirt-shell] sys.stdin.flush() issue on BSD, MacOS | ||
---|---|---|---|
Product: | [oVirt] ovirt-engine-cli | Reporter: | Jiri Belka <jbelka> |
Component: | Core | Assignee: | Juan Hernández <juan.hernandez> |
Status: | CLOSED CURRENTRELEASE | QA Contact: | Jiri Belka <jbelka> |
Severity: | medium | Docs Contact: | |
Priority: | unspecified | ||
Version: | 3.6.2.0 | CC: | bugs, jbelka, juan.hernandez, oourfali, sbonazzo |
Target Milestone: | ovirt-4.0.2 | Flags: | rule-engine:
ovirt-4.0.z+
rule-engine: planning_ack+ juan.hernandez: devel_ack+ pnovotny: testing_ack+ |
Target Release: | 3.6.8.0 | ||
Hardware: | x86_64 | ||
OS: | Other | ||
Whiteboard: | |||
Fixed In Version: | ovirt-engine-cli-3.6.8.0 | Doc Type: | Bug Fix |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2016-08-12 14:31:43 UTC | Type: | Bug |
Regression: | --- | Mount Type: | --- |
Documentation: | --- | CRM: | |
Verified Versions: | Category: | --- | |
oVirt Team: | Infra | RHEL 7.3 requirements from Atomic Host: | |
Cloudforms Team: | --- | Target Upstream Version: | |
Embargoed: |
Description
Jiri Belka
2016-02-01 14:19:00 UTC
Actually I think that we don't need that "flush" at all, as flushing an input stream shouldn't have any effect. Jiri, I verified that just removing that line works file in Linux. Can you check if it works in your BSD/MacOS environment? Without that 'finally' block it runs too. there's no ovirt-engine-cli 4.x rpm, thus the code lives only in git/source but rpms have old code. please move on_qa when a rpm will be ready, thx. [root@jb-rhevm40 ~]# cat /usr/lib/python2.7/site-packages/cli/context.py | sed -n '120,130p' self.settings['ovirt-shell:password'] = getpass.getpass() except KeyboardInterrupt: sys.exit('') except EOFError: sys.exit('') finally: sys.stdin.flush() def __option_error(self, opt): from cli.messages import Messages sys.exit('\n' + Messages.Error.NO_SUCH_OPTION % opt + '\n') [root@jb-rhevm40 ~]# rpm -q ovirt-engine-cli ovirt-engine-cli-3.6.7.0-1.el7ev.noarch [root@jb-rhevm40 ~]# rpm -q ovirt-engine ovirt-engine-4.0.0.6-0.1.el7ev.noarch ok # rpm2cpio ovirt-engine-cli-3.6.8.1-1.el7.centos.noarch.rpm | cpio --to-stdout -i './usr/lib/python2.7/site-packages/cli/context.py' 2>/dev/null | sed -n '120,130p' self.settings['ovirt-shell:password'] = getpass.getpass() except KeyboardInterrupt: sys.exit('') except EOFError: sys.exit('') def __option_error(self, opt): from cli.messages import Messages sys.exit('\n' + Messages.Error.NO_SUCH_OPTION % opt + '\n') def __is_option_specified_in_cli_args(self, optionname): |