Skip to content

Update dependencies for /integrations/kube-agent-updater#26909

Merged
jentfoo merged 3 commits intomasterfrom
jent/integrations_updates
May 25, 2023
Merged

Update dependencies for /integrations/kube-agent-updater#26909
jentfoo merged 3 commits intomasterfrom
jent/integrations_updates

Conversation

@jentfoo
Copy link
Copy Markdown
Contributor

@jentfoo jentfoo commented May 25, 2023

Dependency updates that should apply cleanly for /integrations/kube-agent-updater. This should reduce some of the PR's opened once #26867 is merged. Other PR's don't have CVE's associated to them so we may choose to close those PR's when #26867 is merged.

@jentfoo jentfoo added the dependencies Pull requests that update a dependency file label May 25, 2023
@jentfoo jentfoo self-assigned this May 25, 2023
@github-actions github-actions Bot requested a review from jimbishopp May 25, 2023 16:13
Comment thread integrations/kube-agent-updater/go.mod Outdated
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

The 0.4.0 update required a lot of manual fixes in Teleport. Are we sure no changes are necessary here?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

It does seem that I was optimistic in including everything here, I am working on reducing this change set so that tests will pass

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I left this update in because make test-kube-agent-updater seems to be passing, I will watch for other unit test failures.

Copy link
Copy Markdown
Contributor

@codingllama codingllama left a comment

Choose a reason for hiding this comment

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

Looks like you've got a few breakages.

Also, FYI @hugoShaka.

dependabot Bot added 3 commits May 25, 2023 10:57
Bumps [k8s.io/apimachinery](https://github.com/kubernetes/apimachinery) from 0.26.1 to 0.27.2.
- [Commits](kubernetes/apimachinery@v0.26.1...v0.27.2)

---
updated-dependencies:
- dependency-name: k8s.io/apimachinery
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Bumps [github.com/jonboulle/clockwork](https://github.com/jonboulle/clockwork) from 0.3.0 to 0.4.0.
- [Release notes](https://github.com/jonboulle/clockwork/releases)
- [Commits](jonboulle/clockwork@v0.3.0...v0.4.0)

---
updated-dependencies:
- dependency-name: github.com/jonboulle/clockwork
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Bumps [github.com/stretchr/testify](https://github.com/stretchr/testify) from 1.8.2 to 1.8.3.
- [Release notes](https://github.com/stretchr/testify/releases)
- [Commits](stretchr/testify@v1.8.2...v1.8.3)

---
updated-dependencies:
- dependency-name: github.com/stretchr/testify
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
@jentfoo jentfoo force-pushed the jent/integrations_updates branch from 4cb15ca to bd1093d Compare May 25, 2023 16:58
@jentfoo
Copy link
Copy Markdown
Contributor Author

jentfoo commented May 25, 2023

Thank you @zmb3 and @codingllama. I just updated the PR and the description. It looks like most of these updates introduced one issue or another. I have reduced this to a much smaller set of updates, but which should be safe 🤞

@rosstimothy rosstimothy requested a review from hugoShaka May 25, 2023 17:01
Copy link
Copy Markdown
Contributor

@codingllama codingllama left a comment

Choose a reason for hiding this comment

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

LGTM.

Merged via the queue into master with commit ae22690 May 25, 2023
@jentfoo jentfoo deleted the jent/integrations_updates branch May 25, 2023 23:01
@public-teleport-github-review-bot
Copy link
Copy Markdown

@jentfoo See the table below for backport results.

Branch Result
branch/v13 Failed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file size/sm

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants