Bug #5291
closedDifficult to be sure that a package install has ended
Added by Phillip Davis almost 9 years ago. Updated over 8 years ago.
100%
Description
The Package Installer screen has a scrolling text box that shows the progress of the package installation. Various steps along the way will say:
Something-or-other... done.
So it is difficult to know if there is a "done" displayed, if that is the last "done" and it is really all finished.
The title above that text box continues to say "Beginning package installation".
On 2.2.* that title was a box and the text content would change to things like:
- Package deleted.
- Installation of Notes completed.
It would be good to have some positive feedback when the package install process has really finished.
Files
Package-Installation.png (43.7 KB) Package-Installation.png | Phillip Davis, 10/09/2015 11:41 AM | ||
ReinstallAll.png (28.3 KB) ReinstallAll.png | Phillip Davis, 10/15/2015 10:15 PM | ||
reinstall.jpg (107 KB) reinstall.jpg | Anonymous, 10/16/2015 08:04 AM |
Updated by Phillip Davis almost 9 years ago
Also when you remove a package, the title of the text box says "Beginning package installation" - that is a bit dumb, but I think it did that in 2.2.* also. It should say "Beginning package removal".
At the end of package removal there is some text output that says "Package deleted" - in places like that the word "deleted" could be changed to "removed" to match the button text that the user pushed originally.
Updated by Anonymous almost 9 years ago
- Status changed from New to Confirmed
- Assignee set to Phillip Davis
Duplicates #5279
The package manager is a WIP. I am converting it to a system that starts the installation process in the background and then monitors it via AJAX to get the status and progress. It will be another couple of days before it is ready for testing.
Stay tuned.
Updated by Chris Buechler almost 9 years ago
- Assignee deleted (
Phillip Davis)
part of #5279, but good feedback here, will leave to make sure that's addressed once finished.
Updated by Anonymous almost 9 years ago
- File pkg_install.gif added
- Status changed from Confirmed to Assigned
- The installer is run in the background so that the GUI remains responsive
- The GUI polls for status/progress via AJAX
- The progress bar works correctly, obtaining its information from the installer JSON output
- The exit status of each step is indicated by "Success" or "Failure" in the text window
- A banner is displayed showing "Wait", "Success" and "failed" (paraphrasing). The background color is changed to match the message.
- Handle page reloads and catch some of the less obvious errors.
- Combine the pfSense-upgrade and pfSense-upgrade-GUI.sh scripts
- Add some gettext()-s where needed
Updated by Anonymous almost 9 years ago
- Status changed from Assigned to Feedback
- Assignee changed from Anonymous to Phillip Davis
The package installer now shows a color coded completion banner, "real time" log data and a progress bar.
The progress bar is of limited use at them moment since some packages (pfBlockerNG is an example) run for close to a minute, but only output progress information during the 1st second. Perhaps this can be improved too.
Updated by Anonymous almost 9 years ago
- % Done changed from 0 to 100
Applied in changeset pfsense:35fcf1c1eb105da320b63550edfa5063af7312b3.
Updated by Phillip Davis almost 9 years ago
See pull request https://github.com/pfsense/pfsense/pull/1965 for a couple of changes to tidy up a few bits of text.
Works nicely, thanks.
Note that "reinstallall" is not working - should I raise another redmine report for that? Or is it already a known part of WIP?
Updated by Anonymous almost 9 years ago
I believe the reinstallation issue was fixed late yesterday with an update to the pfSense-update script but I will check. Thanks for the PR. I'll look at it later today.
Updated by Anonymous almost 9 years ago
Confirmed that reinstallation works as expected. (tested with sudo, snort and pfBlockerNG)
PR merged and tested.
Thanks!
Updated by Phillip Davis almost 9 years ago
Single package reinstall works.
But Diagnostics->Backup/Restore, "Reinstall Packages" does nothing after pressing "Confirm".
Updated by Anonymous almost 9 years ago
reinstallall functionality restored and tested.
Please note that the progress bar is suppressed during this operation. It is not really possible to know in advance how many steps are required, so it would jump about in a nonsensical manner. The textarea tells all and the status banner information is accurate.
Updated by Phillip Davis almost 9 years ago
- File ReinstallAll.png ReinstallAll.png added
I did reinstall all on a system with Cron and OpenVPN Client Export Utility installed. The reinstall completed very quickly and the text area has just the word "Success".
Is it supposed to be that quiet?
I tried editing /usr/local/www/packages/cron/cron.php - just put some rubbish text in comments. Then do package reinstall all. The edited version of cron.php was still there. I expected that the reinstall would overwrite all the files supplied by the package with the proper package versions.
What is really supposed to happen?
I did an individual reinstall of Cron package. cron.php went back to the original code as I expected.
Updated by Anonymous almost 9 years ago
No. You should see the logs from the uninstall and install processes for each package. I tested with a VM and with real hardware but perhaps there is some timing issue. I'll try it on a few more. platforms and with the same packages you tested with.
Would you mind taking a look in /tmp/webgui-log.txt please? That should contain the logs.
Updated by Anonymous almost 9 years ago
Applied in changeset pfsense:1166ee1e443ac91ea827a567ab40975e0ec8213e.
Updated by Anonymous almost 9 years ago
- File reinstall.jpg reinstall.jpg added
Never mind the logs.
There were two issues specific to pfSense-pkg-openvpn-client-export:
My "Does the string begin with pfSense-pkg-" trick was crude and failed on all of the '-' in this package name, and once installed you cannot access the functionality because a required PHP function has been removed.
I have fixed the former (you should see more sensible re-install all output now) and have sent a note to Renato about the latter.
Thanks again for all of you testing. It really helps :)
Updated by Anonymous almost 9 years ago
openvpn client export issue moved to a new ticket #5311
Updated by Phillip Davis almost 9 years ago
General install, remove, reinstall and reinstall all seems good now. I just tried all the combinations on:
2.3-ALPHA (amd64)
built on Mon Oct 26 19:32:58 CDT 2015
FreeBSD 10.2-STABLE
The functionality works and the messages along the way are consistent with what is happening. IMHO this is resolved.
Updated by Renato Botelho over 8 years ago
- Project changed from Bootstrap to pfSense
Updated by Chris Buechler over 8 years ago
- Category set to Package System
- Status changed from Feedback to Resolved
fixed