Skip to content
This repository has been archived by the owner on Dec 4, 2024. It is now read-only.

traefik: bump to chart 1.72.19 #259

Merged
merged 2 commits into from
May 21, 2020
Merged

traefik: bump to chart 1.72.19 #259

merged 2 commits into from
May 21, 2020

Conversation

jieyu
Copy link
Contributor

@jieyu jieyu commented May 21, 2020

What type of PR is this?

Bug

What this PR does/ why we need it:

The chart change improves the charts by allowing extra SSL entrypoints
being defined, and inherit SSL configurations from global SSL config. We
observed that in some customer environment, they are using advanced
configurations for their ingress, including turning on proxy protocol on
the ingress.

Previously, we use extraConfigEntrypoints to add the additional velero
minio entrypoints for the velero addon. This basically means if they
have any customization on the ingress entrypoint, we'll have to ask them
to manually specifying those configs extraConfigEntrypoints again for
the velero-minio entrypoint, which creates a very bad user experience.

This chart change improves this experiecne by introducing
extraSSLEntrypoints that will inherit global configurations so that
the user does not have to do anything on the additional entrypoints.

This patch update the Addon config accordingly to use the new
extraSSLEntrypoints.

Which issue(s) this PR fixes:

https://jira.d2iq.com/browse/D2IQ-68705

Special notes for your reviewer:

Does this PR introduce a user-facing change?:

[traefik] fix the velero-minio entrypoint to inherit global ssl and proxy protocol configurations 

Checklist

  • The commit message explains the changes and why are needed.
  • The code builds and passes lint/style checks locally.
  • The relevant subset of integration tests pass locally.
  • The core changes are covered by tests.
  • The documentation is updated where needed.

jieyu added 2 commits May 21, 2020 09:51
To prepare for upgrade.
The chart change improves the charts by allowing extra SSL entrypoints
being defined, and inherit SSL configurations from global SSL config. We
observed that in some customer environment, they are using advanced
configurations for their ingress, including turning on proxy protocol on
the ingress.

Previously, we use `extraConfigEntrypoints` to add the additional velero
minio entrypoints for the velero addon. This basically means if they
have any customization on the ingress entrypoint, we'll have to ask them
to manually specifying those configs `extraConfigEntrypoints` again for
the velero-minio entrypoint, which creates a very bad user experience.

This chart change improves this experiecne by introducing
`extraSSLEntrypoints` that will inherit global configurations so that
the user does not have to do anything on the additional entrypoints.

This patch update the Addon config accordingly to use the new
`extraSSLEntrypoints`.
@jieyu jieyu requested a review from a team as a code owner May 21, 2020 17:03
@jieyu jieyu requested a review from GoelDeepak May 21, 2020 17:03
@jieyu
Copy link
Contributor Author

jieyu commented May 21, 2020

The AWS failure seems to be a flake due to the underlying infrastructure issue.

Created https://jira.d2iq.com/browse/D2IQ-68751 to track

Copy link
Contributor

@alejandroEsc alejandroEsc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@alejandroEsc alejandroEsc merged commit ced6f75 into master May 21, 2020
@alejandroEsc alejandroEsc deleted the fix_traefik branch May 21, 2020 21:43
@samvantran samvantran mentioned this pull request Jun 10, 2020
joejulian pushed a commit that referenced this pull request Jun 19, 2020
* traefik: copy from previous revision

To prepare for upgrade.

* traefik: bump to chart 1.72.19

The chart change improves the charts by allowing extra SSL entrypoints
being defined, and inherit SSL configurations from global SSL config. We
observed that in some customer environment, they are using advanced
configurations for their ingress, including turning on proxy protocol on
the ingress.

Previously, we use `extraConfigEntrypoints` to add the additional velero
minio entrypoints for the velero addon. This basically means if they
have any customization on the ingress entrypoint, we'll have to ask them
to manually specifying those configs `extraConfigEntrypoints` again for
the velero-minio entrypoint, which creates a very bad user experience.

This chart change improves this experiecne by introducing
`extraSSLEntrypoints` that will inherit global configurations so that
the user does not have to do anything on the additional entrypoints.

This patch update the Addon config accordingly to use the new
`extraSSLEntrypoints`.
joejulian pushed a commit that referenced this pull request Jul 22, 2020
* traefik: copy from previous revision

To prepare for upgrade.

* traefik: bump to chart 1.72.19

The chart change improves the charts by allowing extra SSL entrypoints
being defined, and inherit SSL configurations from global SSL config. We
observed that in some customer environment, they are using advanced
configurations for their ingress, including turning on proxy protocol on
the ingress.

Previously, we use `extraConfigEntrypoints` to add the additional velero
minio entrypoints for the velero addon. This basically means if they
have any customization on the ingress entrypoint, we'll have to ask them
to manually specifying those configs `extraConfigEntrypoints` again for
the velero-minio entrypoint, which creates a very bad user experience.

This chart change improves this experiecne by introducing
`extraSSLEntrypoints` that will inherit global configurations so that
the user does not have to do anything on the additional entrypoints.

This patch update the Addon config accordingly to use the new
`extraSSLEntrypoints`.
joejulian pushed a commit that referenced this pull request Jul 22, 2020
* traefik: copy from previous revision

To prepare for upgrade.

* traefik: bump to chart 1.72.19

The chart change improves the charts by allowing extra SSL entrypoints
being defined, and inherit SSL configurations from global SSL config. We
observed that in some customer environment, they are using advanced
configurations for their ingress, including turning on proxy protocol on
the ingress.

Previously, we use `extraConfigEntrypoints` to add the additional velero
minio entrypoints for the velero addon. This basically means if they
have any customization on the ingress entrypoint, we'll have to ask them
to manually specifying those configs `extraConfigEntrypoints` again for
the velero-minio entrypoint, which creates a very bad user experience.

This chart change improves this experiecne by introducing
`extraSSLEntrypoints` that will inherit global configurations so that
the user does not have to do anything on the additional entrypoints.

This patch update the Addon config accordingly to use the new
`extraSSLEntrypoints`.
joejulian pushed a commit that referenced this pull request Jul 23, 2020
* traefik: copy from previous revision

To prepare for upgrade.

* traefik: bump to chart 1.72.19

The chart change improves the charts by allowing extra SSL entrypoints
being defined, and inherit SSL configurations from global SSL config. We
observed that in some customer environment, they are using advanced
configurations for their ingress, including turning on proxy protocol on
the ingress.

Previously, we use `extraConfigEntrypoints` to add the additional velero
minio entrypoints for the velero addon. This basically means if they
have any customization on the ingress entrypoint, we'll have to ask them
to manually specifying those configs `extraConfigEntrypoints` again for
the velero-minio entrypoint, which creates a very bad user experience.

This chart change improves this experiecne by introducing
`extraSSLEntrypoints` that will inherit global configurations so that
the user does not have to do anything on the additional entrypoints.

This patch update the Addon config accordingly to use the new
`extraSSLEntrypoints`.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants