microsoft / oxa-tools

Open edX on Azure Tools
MIT License
37 stars 99 forks source link

Add support for mandatory installation of latest OMI #324

Closed eltoncarr closed 6 years ago

eltoncarr commented 6 years ago

What does this PR do? Please provide some context

Add support for installing the latest version of OMI package.

Reason [From Azure] A bug was found related to the crontab entry recently, and a hot fix, omi-1.4.1-1, was released. The fix is also available in most recent release, omi-1.4.2-1. This bug occurred on systems that don’t have Kerberos installed, and caused an error email to be generated every minute.

Where should the reviewer start?

The updated utilities.sh library file and how it is integrated with the installation process via the install-tools function.

How can this be manually tested? (brief repro steps and corpnet-URL with change)

source utilities.sh install-omi

What are the relevant TFS items? (list id numbers)

152377

Definition of done:

Reminders DURING merge

  1. If you're merging from a short-term (feature) branch into a long-term branch (like dev, release, or master) then "Squash and merge" to keep our history clean.
  2. If merging from two longterm branches (like cherry picks from upstream, dev to release, etc) then "Create merge commit" to preserve individual commits.