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.

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

docs: update requirements (urllib3 1.26.5, requests 2.25.1) #16396

Merged
merged 1 commit into from Jun 3, 2021

Conversation

qmonnet
Copy link
Member

@qmonnet qmonnet commented Jun 2, 2021

In #16390, dependabot says:

build(deps): bump urllib3 from 1.24.2 to 1.26.5 in /Documentation

- [Release notes](https://github.com/urllib3/urllib3/releases)
- [Changelog](https://github.com/urllib3/urllib3/blob/main/CHANGES.rst)
- [Commits](urllib3/urllib3@1.24.2...1.26.5)

---
updated-dependencies:
- dependency-name: urllib3
  dependency-type: direct:production
...

However, this breaks the build for the documentation, because:

requests 2.20.0 depends on urllib3<1.25 and >=1.21.1

Let's update the version for urllib3, but let's also update requests to its latest version.

In cilium#16390, dependabot says:

    build(deps): bump urllib3 from 1.24.2 to 1.26.5 in /Documentation

    - [Release notes](https://github.com/urllib3/urllib3/releases)
    - [Changelog](https://github.com/urllib3/urllib3/blob/main/CHANGES.rst)
    - [Commits](urllib3/urllib3@1.24.2...1.26.5)

    ---
    updated-dependencies:
    - dependency-name: urllib3
      dependency-type: direct:production
    ...

However, this breaks the build for the documentation, because:

    requests 2.20.0 depends on urllib3<1.25 and >=1.21.1

Let's update the version for urllib3, but let's also update "requests"
to its latest version.

Signed-off-by: Quentin Monnet <quentin@isovalent.com>
@qmonnet qmonnet added area/documentation Impacts the documentation, including textual changes, sphinx, or other doc generation code. release-note/misc This PR makes changes that have no direct user impact. labels Jun 2, 2021
@qmonnet qmonnet requested a review from a team as a code owner June 2, 2021 14:10
@qmonnet qmonnet requested a review from joestringer June 2, 2021 14:10
@qmonnet qmonnet added the ready-to-merge This PR has passed all tests and received consensus from code owners to merge. label Jun 3, 2021
@nathanjsweet nathanjsweet merged commit 05b4e6c into cilium:master Jun 3, 2021
@qmonnet qmonnet deleted the pr/update_doc_reqs branch June 3, 2021 14:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/documentation Impacts the documentation, including textual changes, sphinx, or other doc generation code. ready-to-merge This PR has passed all tests and received consensus from code owners to merge. release-note/misc This PR makes changes that have no direct user impact.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants