Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Bug 1714140: fix(catalog): re-install resources in existing installplan #965

Merged
merged 2 commits into from
Jul 31, 2019
Merged

Bug 1714140: fix(catalog): re-install resources in existing installplan #965

merged 2 commits into from
Jul 31, 2019

Commits on Jul 26, 2019

  1. fix(catalog): re-install resources in existing installplan 

    This specifically fixes the scenario where a subscription is deleted and
recreated in a namespace with an additional subscription. When the
subscription of interest is deleted, the associated installplan remains
due to the other subscription's owner reference. Because the existing
installplan manifests match, no additional work is done. This is a
problem if resources that were originally installed have been deleted.

Now after a subscription is recreated, the proper owner references are
re-added onto the install plan as well as all the resources are checked
that they are installed.
    @jpeeler
    jpeeler committed Jul 26, 2019
    Configuration menu
    Copy the full SHA
    7f6da18 View commit details
    Browse the repository at this point in the history

  2. fix(unit): allow logrus log level to be adjusted 

    Using the standard logger allows a unit test to set the desired log
level via `log.SetLevel(log.DebugLevel)`.
    @jpeeler
    jpeeler committed Jul 26, 2019
    Configuration menu
    Copy the full SHA
    4af534f View commit details
    Browse the repository at this point in the history