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

Add report-status & leader-election to Helm chart #335

Merged
merged 8 commits into from
Aug 16, 2018
Merged

Add report-status & leader-election to Helm chart #335

merged 8 commits into from
Aug 16, 2018

Commits on Aug 16, 2018

  1. Helm: Add report-status & leader-election support

    @Dean-Coakley
    Dean-Coakley authored and isaac committed Aug 16, 2018
    Configuration menu
    Copy the full SHA
    b3a6f6e View commit details
    Browse the repository at this point in the history

  2. Helm: Fix external-service cli argument templates

    @Dean-Coakley
    Dean-Coakley authored and isaac committed Aug 16, 2018
    Configuration menu
    Copy the full SHA
    65391c1 View commit details
    Browse the repository at this point in the history

  3. Helm: Add report-ingress-status cli arg 

    * Added report-ingress-status to both templates
* Fixed leader-election to only be set when report-ingress-status is
enabled
    @Dean-Coakley
    Dean-Coakley authored and isaac committed Aug 16, 2018
    Configuration menu
    Copy the full SHA
    daa9824 View commit details
    Browse the repository at this point in the history

  4. Helm: Fix defaults for reporting status 

    * Fix external-status-address to use example ip 1.2.3.4
* Fix externalService default to be nginx-ingress as this will be the
most common default
* Removed quotes from externalService in values files and moved to
templates to follow existing code style
    @Dean-Coakley
    Dean-Coakley authored and isaac committed Aug 16, 2018
    Configuration menu
    Copy the full SHA
    eb6e943 View commit details
    Browse the repository at this point in the history

  5. Helm: Update README with new options 

    * Added documentation for report-ingress-status related values
* Added documentation for leader-election
    @Dean-Coakley
    Dean-Coakley authored and isaac committed Aug 16, 2018
    Configuration menu
    Copy the full SHA
    1661f49 View commit details
    Browse the repository at this point in the history

  6. Helm: Fix report-status/leader-election docs

    @Dean-Coakley
    Dean-Coakley authored and isaac committed Aug 16, 2018
    Configuration menu
    Copy the full SHA
    f7fea6b View commit details
    Browse the repository at this point in the history

  7. Selectively enable permissions for status update 

    The IC only requires ingress/status update permissions if
report-ingress-status is enabled.
    isaac committed Aug 16, 2018
    Configuration menu
    Copy the full SHA
    c1e723a View commit details
    Browse the repository at this point in the history

  8. Enable report-ingress-status by default 

    Also add get permissions on ingress resources, needed when the status
updater is retrying a status update.
    isaac committed Aug 16, 2018
    Configuration menu
    Copy the full SHA
    46500c1 View commit details
    Browse the repository at this point in the history