First pull request:
with another to follow to add database backup functionality.
Currently have had the following merged:
The first sets up work following patches that have been opened up:
The latter of those two allows the `appliance_console` trigger database dumps, and the former was already in place for work that was done previously before requirements changed.
The changes to `manageiq-gems-pending` sets up the work for:
Which is required to allow the DB Dumps to work in the `appliance_console` (backups are already supported rake tasks, so the change to allow backups in the `appliance_console` needed no supporting PRs).
No option added to appliance_console menu for taking backups.
This should not have been moved to POST as we have not released a new version of the appliance console gem with this fix.
NickL, do we want to cut a version now or are there more changes coming in soon?
There are more changes coming that I think make sense to be released together.
Moving back to POST for the time being.
I have added a bugfix PR since the previous round of merges:
Which fixes some major usability flaws with the initial implementation which were not caught in tests or code review (tests have been updated to catch this better going forward).
New commit detected on ManageIQ/manageiq-appliance/master:
Author: Nick Carboni <email@example.com>
AuthorDate: Wed Aug 1 16:53:52 2018 -0400
Commit: Nick Carboni <firstname.lastname@example.org>
CommitDate: Wed Aug 1 16:53:52 2018 -0400
Bump versions of the console and HA admin gem
manageiq-appliance-dependencies.rb | 4 +-
1 file changed, 2 insertions(+), 2 deletions(-)
New commit detected on ManageIQ/manageiq-appliance_console/master:
Author: Keenan Brock <email@example.com>
AuthorDate: Mon Sep 24 12:43:59 2018 -0400
Commit: Keenan Brock <firstname.lastname@example.org>
CommitDate: Mon Sep 24 12:43:59 2018 -0400
anonymous ftp upload option
This will allow customers to upload files to support sites
To add this option, add to `en.yml`:
local: Local file
filename_text: "The case number dash (-) filename. (e.g.: 12345-db.backup)"
lib/manageiq/appliance_console/database_admin.rb | 41 +-
spec/database_admin_spec.rb | 101 +
2 files changed, 139 insertions(+), 3 deletions(-)
I cannot find the option "Create Database Backup" in the appliance_console menu on
Sorry for missing the ping on this.
That said, confused by the most recent comment, as this is adding "Include option to take database dump", and you have mentioned that you can't find the "Create Database Backup" option.
I will be checking against the the Hammer beta build in a bit, but wanted to at least confirm you got the right ticket in regards to your most recent comment. There are some bugs that I am currently aware of and in the process of fixing, but a missing menu item shouldn't be one of them.
Also, just for reference, these should be part of the build as of:
So this might be more of a build issue than on my end. Again, I am checking against a Hammer appliance to validate one way or the other.
Here is the menu that I got when using the Vagrant build for Hammer beta2:
1) Configure Network
2) Set Timezone
3) Set Date and Time
4) Create Database Backup
5) Create Database Dump
6) Restore Database From Backup
7) Configure Database
8) Configure Database Replication
9) Logfile Configuration
10) Configure Application Database Failover Monitor
11) Configure External Authentication (httpd)
12) Update External Authentication Options
13) Generate Custom Encryption Key
14) Stop EVM Server Processes
15) Start EVM Server Processes
16) Restart Appliance
17) Shut Down Appliance
18) Summary Information
Choose the advanced setting:
So dump, backup, and restore are all present, so I am going to chalk it up to the appliance was missing the dependency when this was tested previously.
Assigning this back to QA for re-evaluation.
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.
For information on the advisory, and where to find the updated
files, follow the link below.
If the solution does not work for you, open a new bug report.