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

[external-dns] chore: change chart location #601

Merged
merged 2 commits into from
Oct 20, 2020
Merged

Conversation

alejandroEsc
Copy link
Contributor

What type of PR is this?

Chore

What this PR does/ why we need it:

Which issue(s) this PR fixes:

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

Special notes for your reviewer:

Does this PR introduce a user-facing change?:

NONE

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.

addons/external-dns/external-dns.yaml Outdated Show resolved Hide resolved
addons/external-dns/external-dns.yaml Outdated Show resolved Hide resolved
Copy link
Contributor

@shaneutt shaneutt left a comment

Choose a reason for hiding this comment

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

LGTM, but before we merge this let's please do a manual check just to make sure the chart is behaving exactly as we intend.

@alejandroEsc alejandroEsc requested a review from a team as a code owner October 20, 2020 16:55
@alejandroEsc
Copy link
Contributor Author

The TC test passed, though it's not reporting.

@alejandroEsc alejandroEsc merged commit dc1785f into master Oct 20, 2020
@alejandroEsc alejandroEsc deleted the ae/external-dns branch October 20, 2020 20:23
alejandroEsc added a commit that referenced this pull request Oct 22, 2020
* Update CODEOWNERS (#606)

* [prometheus-adapter] chore: change chart origin (#603)

* [prometheus-adapter] chore: change chart origin

* change revision version

* Update addons/prometheusadapter/prometheusadapter.yaml

Co-authored-by: Grace Do <[email protected]>

* fix chart name

Co-authored-by: Grace Do <[email protected]>

* [external-dns] chore: change chart location (#601)

* [external-dns] chore: change chart location

* fix issue with chart name

* chore: bump cert-manager

* chore: Bump revision and values

Co-authored-by: Shane Utt <[email protected]>
Co-authored-by: Alejandro Escobar <[email protected]>
Co-authored-by: Grace Do <[email protected]>
joejulian added a commit that referenced this pull request Dec 15, 2020
* feat: bump for cert-manager-upgrade (#594)

* feat: bump for cert-manager-upgrade

* fix: use the correct image

* use log for log output instead of stdout

* Update addons/cert-manager/cert-manager.yaml

Co-authored-by: Grace Do <[email protected]>

* Update test/scripts/test-wrapper.go

Co-authored-by: Grace Do <[email protected]>

Co-authored-by: Joe Julian <[email protected]>
Co-authored-by: Grace Do <[email protected]>

* chore: bump cert-manager (#607)

* Update CODEOWNERS (#606)

* [prometheus-adapter] chore: change chart origin (#603)

* [prometheus-adapter] chore: change chart origin

* change revision version

* Update addons/prometheusadapter/prometheusadapter.yaml

Co-authored-by: Grace Do <[email protected]>

* fix chart name

Co-authored-by: Grace Do <[email protected]>

* [external-dns] chore: change chart location (#601)

* [external-dns] chore: change chart location

* fix issue with chart name

* chore: bump cert-manager

* chore: Bump revision and values

Co-authored-by: Shane Utt <[email protected]>
Co-authored-by: Alejandro Escobar <[email protected]>
Co-authored-by: Grace Do <[email protected]>

* bump prometheus-operator chart

* Add a new label for kommander to require (#655)

Signed-off-by: Dimitri Koshkin <[email protected]>

* feat: adds cert-manager-upgrade automation (#656)

* cert manager v0.4.1 (#659)

* fix: add change to fix kubefed certificate

* chore: bump revision

* fix: ensure organization is set (#662)

* add requires cert-manager: v1 (#664)

* add requires cert-manager: v1

* add cert-manager requires for dex

* [cert-manager] chart bump (#674)

* chart bump

* update v ref

* 3.0.0 Release Notes (#676)

* Release notes for 3.0.0

* fix: use chart version that does not contain patch (#722)

Co-authored-by: Jared Rodriguez <[email protected]>
Co-authored-by: Grace Do <[email protected]>
Co-authored-by: Hector Fernandez <[email protected]>
Co-authored-by: Shane Utt <[email protected]>
Co-authored-by: Alejandro Escobar <[email protected]>
Co-authored-by: Dimitri Koshkin <[email protected]>
Co-authored-by: Steve Chung <[email protected]>
gracedo pushed a commit that referenced this pull request Dec 22, 2020
* [external-dns] chore: change chart location

* fix issue with chart name
gracedo pushed a commit that referenced this pull request Dec 22, 2020
* [external-dns] chore: change chart location

* fix issue with chart name
joejulian added a commit that referenced this pull request Jan 12, 2021
* feat: bump for cert-manager-upgrade (#594)

* feat: bump for cert-manager-upgrade

* fix: use the correct image

* use log for log output instead of stdout

* Update addons/cert-manager/cert-manager.yaml

Co-authored-by: Grace Do <[email protected]>

* Update test/scripts/test-wrapper.go

Co-authored-by: Grace Do <[email protected]>

Co-authored-by: Joe Julian <[email protected]>
Co-authored-by: Grace Do <[email protected]>

* chore: bump cert-manager (#607)

* Update CODEOWNERS (#606)

* [prometheus-adapter] chore: change chart origin (#603)

* [prometheus-adapter] chore: change chart origin

* change revision version

* Update addons/prometheusadapter/prometheusadapter.yaml

Co-authored-by: Grace Do <[email protected]>

* fix chart name

Co-authored-by: Grace Do <[email protected]>

* [external-dns] chore: change chart location (#601)

* [external-dns] chore: change chart location

* fix issue with chart name

* chore: bump cert-manager

* chore: Bump revision and values

Co-authored-by: Shane Utt <[email protected]>
Co-authored-by: Alejandro Escobar <[email protected]>
Co-authored-by: Grace Do <[email protected]>

* bump prometheus-operator chart

* Add a new label for kommander to require (#655)

Signed-off-by: Dimitri Koshkin <[email protected]>

* feat: adds cert-manager-upgrade automation (#656)

* cert manager v0.4.1 (#659)

* fix: add change to fix kubefed certificate

* chore: bump revision

* fix: ensure organization is set (#662)

* add requires cert-manager: v1 (#664)

* add requires cert-manager: v1

* add cert-manager requires for dex

* [cert-manager] chart bump (#674)

* chart bump

* update v ref

* 3.0.0 Release Notes (#676)

* Release notes for 3.0.0

* fix: use chart version that does not contain patch (#722)

Co-authored-by: Jared Rodriguez <[email protected]>
Co-authored-by: Grace Do <[email protected]>
Co-authored-by: Hector Fernandez <[email protected]>
Co-authored-by: Shane Utt <[email protected]>
Co-authored-by: Alejandro Escobar <[email protected]>
Co-authored-by: Dimitri Koshkin <[email protected]>
Co-authored-by: Steve Chung <[email protected]>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants