Bug 1379146 - HE install via CockPit cut the install text allow only a few dozen lines of history.
Summary: HE install via CockPit cut the install text allow only a few dozen lines of h...
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: ovirt-node
Classification: oVirt
Component: UI
Version: 4.0
Hardware: Unspecified
OS: Unspecified
high
high
Target Milestone: ovirt-4.1.0-beta
: 4.1
Assignee: Ryan Barry
QA Contact: Yihui Zhao
URL:
Whiteboard:
Depends On: 1367457
Blocks:
TreeView+ depends on / blocked
 
Reported: 2016-09-25 12:21 UTC by Yaniv Lavi
Modified: 2017-02-01 14:50 UTC (History)
3 users (show)

Fixed In Version: cockpit-ovirt-0.10.6-1.4.3.el7ev
Doc Type: Bug Fix
Doc Text:
Cause: To limiting the number of alerts from hosted-engine-setup, the number of messages visible simultaneously was limited. Consequence: Normal output from hosted-engine-setup was accidentally truncated. Fix: Normal output is now screened for. Result: User-visible output from hosted-engine-setup is not limited.
Clone Of:
Environment:
Last Closed: 2017-02-01 14:50:15 UTC
oVirt Team: Node
Embargoed:
rule-engine: ovirt-4.1+
rule-engine: planning_ack+
rule-engine: devel_ack+
cshao: testing_ack+


Attachments (Terms of Use)
screenshot (118.96 KB, image/png)
2016-09-26 00:15 UTC, Ryan Barry
no flags Details


Links
System ID Private Priority Status Summary Last Updated
oVirt gerrit 68755 0 ovirt-4.1 MERGED dashboard: don't truncate hosted engine output 2016-12-20 15:28:19 UTC
oVirt gerrit 68833 0 ovirt-4.1 POST dashboard: don't truncate hosted engine output 2016-12-20 15:28:12 UTC

Description Yaniv Lavi 2016-09-25 12:21:00 UTC
Description of problem:
HE install via CockPit cut the install text allow only a few dozen lines of history. This can cause issue when for example user wants to access console of the VM with password displayed in the setup process.

Version-Release number of selected component (if applicable):
4.0

How reproducible:
Always

Steps to Reproduce:
1. Run HE install via CockPit
2. Try to access console in late stages of setup

Actual results:
Unable to scroll back to that section of setup.

Expected results:
Should allow scrolling to any part of the setup process.

Additional info:

Comment 1 Ryan Barry 2016-09-25 21:58:23 UTC
There is almost no chance this will be fixed in 4.0.z. It involves a significant amount of development work on the back end, and a redesign is happening in 4.1

Please file an RFE against 4.1 to keep the full log somewhere accessible.

Fabian, please nack this until when/if there's a customer ticket.

Comment 2 Ryan Barry 2016-09-25 22:03:53 UTC
Also, please note that the output window is only truncated once input is submitted. Any/all output is kept in-between input submissions.

Additionally, my understanding is that we are moving to an appliance-only flow, so the console password should be available anyway (since it is a one-stage setup then), assuming cloud-init is not used, which is also the suggested flow.

Comment 3 Fabian Deutsch 2016-09-25 22:07:57 UTC
I should have checked, I assumed that the truncation happens to the text which is shown after a submission.
Which effectively sounded like the text of the previous step is truncated - and this sounded like it could be fixed.

As this is not the case according to comment 1, then I'm moving it out.
And it might rather be implicitly fixed by bug 1367457.

Comment 4 Ryan Barry 2016-09-25 23:14:37 UTC
No worries about checking. We do truncate the info messages after 10 (since the actual VM setup has so many that the would cover the entire screen), so possibly you were thinking of that.

We truncate the messages from the previous step on input, but keep the entire output until input happens.

We initially spoke about keeping the entire log, which is something we should do for 4.1 (particularly as these messages won't be shown again with the US redesign), but the development effort doesn't seem worth it for 4.0.z with a redesign coming, since the code would be abandoned after that.

The logs are still available on the actual host as a workaround if necessary, and we can fix this with ~4-6 hours of development, but probably not worth it unless there's a customer case.

Comment 5 Ryan Barry 2016-09-26 00:15:23 UTC
Created attachment 1204632 [details]
screenshot

Comment 6 Ryan Barry 2016-09-26 00:16:46 UTC
I just attached a screenshot.

We truncate the top, but we don't truncate the bottom.

However, it looks like there was a behavior change at some point (the bottom used to have a scrollbar), possibly in the patternfly component from npm.

We don't truncate this, but it's not possible to scroll up, which should be fixed in 4.0.z...

That's just a UX change, and it's a regression

Comment 7 Yihui Zhao 2016-12-30 10:07:56 UTC
On RHVH 4.0,it was unable to scroll back to that section of setup.
Tested on RHVH 4.1-20161222.0,
if select execte engine-setup automatically on cockpit ,it can scroll to any part of the setup process on cockpit console.

Thanks,
Yihui

Comment 8 Yihui Zhao 2017-01-06 02:03:58 UTC
From Comment7:
When select execte engine-setup automatically on cockpit,it can scroll back to any
part of the setup process on cockpit console.

Version-Release number of selected component (if applicable):
RHVH 4.1

So it work well on RHVH4.1.

Thanks,
Yihui

Comment 9 Yihui Zhao 2017-01-06 03:00:23 UTC
(In reply to Yihui Zhao from comment #8)
> From Comment7:
> When select execte engine-setup automatically on cockpit,it can scroll back
> to any
> part of the setup process on cockpit console.
> 
> Version-Release number of selected component (if applicable):
> RHVH 4.1
> 
> So it work well on RHVH4.1.
> 
> Thanks,
> Yihui


Additional info:
Version-Release number of selected component (if applicable):
rhvh-4.1-0.20170104.0+1
cockpit-ovirt-dashboard-0.10.7-0.0.3.el7ev.noarch
cockpit-ws-126-1.el7.x86_64
cockpit-shell-126-1.el7.noarch
cockpit-storaged-126-1.el7.noarch
cockpit-bridge-126-1.el7.x86_64
ovirt-hosted-engine-ha-2.1.0-0.0.master.git118000f.el7ev.noarch
ovirt-hosted-engine-setup-2.1.0-0.0.master.git46cacd3.el7ev.noarch
ovirt-host-deploy-1.6.0-0.2.master.gitb76ad50.el7ev.noarch
20170103.0-1.el7ev.4.0.rpm( ovirt-engine-appliance rpm)


Note You need to log in before you can comment on or make changes to this bug.