Skip to content

add qb package to Docker#3737

Merged
gazpachoking merged 8 commits intoFlexget:developfrom
trim21:patch-1
Apr 11, 2023
Merged

add qb package to Docker#3737
gazpachoking merged 8 commits intoFlexget:developfrom
trim21:patch-1

Conversation

@trim21
Copy link
Copy Markdown
Contributor

@trim21 trim21 commented Apr 9, 2023 •
edited
Loading

Motivation for changes:

I'm using both qb and transmission…

Detailed changes:

  • add qbittorrent-api python package to docker image

Addressed issues/feature requests:

  • Fixes # .

Config usage if relevant (new plugin or updated schema):

paste_config_here

Log and/or tests output (preferably both):

paste output here

To Do:

  • Stuff..

@trim21 trim21 marked this pull request as ready for review April 9, 2023 21:17
@trim21
Copy link
Copy Markdown
Contributor Author

trim21 commented Apr 9, 2023

looks like dockerfile is not actually tested

@gazpachoking
Copy link
Copy Markdown
Member

gazpachoking commented Apr 10, 2023

I'm good with this.
It made me start thinking though, should we add a dependency group to the pyproject for optional deps that are included in the docker image rather than putting them directly in the dockerfile? Would mean that poetry could resolve the specific versions that get installed and make sure they are compatible with our other deps. @paranoidi @X-dark @ksurl ?

@ksurl
Copy link
Copy Markdown
Contributor

ksurl commented Apr 10, 2023

does that mean it will install at runtime? if it can pin versions rather than latest, it might be ok. unpredictable packages is against the philosophy of static images, but I'm sure other projects break this rule all the time if it works and is limited in scope.

@gazpachoking
Copy link
Copy Markdown
Member

gazpachoking commented Apr 10, 2023

No, not proposing it installs at runtime. We could either have it generate a docker-requirements.txt similar to how we have other requirements files (which the dockerfile would then use) or we could switch the dockerfile to install flexget with poetry ratther than pip. In any case, the deps versions would still be resolved at image creation time.

@gazpachoking
Copy link
Copy Markdown
Member

gazpachoking commented Apr 10, 2023

looks like dockerfile is not actually tested

Yeah. Do you (or anyone) have a good idea of what we should test/how we should test it?

@trim21
Copy link
Copy Markdown
Contributor Author

trim21 commented Apr 10, 2023

looks like dockerfile is not actually tested

Yeah. Do you (or anyone) have a good idea of what we should test/how we should test it?

build and run flexget --help

@ksurl
Copy link
Copy Markdown
Contributor

ksurl commented Apr 10, 2023 •
edited
Loading

No, not proposing it installs at runtime. We could either have it generate a docker-requirements.txt similar to how we have other requirements files (which the dockerfile would then use) or we could switch the dockerfile to install flexget with poetry ratther than pip. In any case, the deps versions would still be resolved at image creation time.

Haha I actually made a local branch to pr a separate requirement file for docker but never pushed it.

ksurl@b0f3cc6

@trim21
Copy link
Copy Markdown
Contributor Author

trim21 commented Apr 10, 2023

I'll just let poetry to export a requiremenets-docker.txt to be requirements.txt with docker group

@trim21
Copy link
Copy Markdown
Contributor Author

trim21 commented Apr 10, 2023

and why we install nodejs in docker image? which plugin need it?

gazpachoking
gazpachoking reviewed Apr 10, 2023
View reviewed changes
@trim21 trim21 requested a review from gazpachoking April 10, 2023 21:51
gazpachoking
gazpachoking reviewed Apr 11, 2023
View reviewed changes
.github/workflows/docker-image.yml
file: ./Dockerfile
builder: ${{ steps.buildx.output.name }}
push: true
push: ${{ github.event_name == 'push' && github.ref != 'refs/heads/master' }}
Copy link
Copy Markdown
Member

@gazpachoking gazpachoking Apr 11, 2023

Choose a reason for hiding this comment

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

What's the motivation for this change? Does it change anything functionally?

Copy link
Copy Markdown
Contributor

@ksurl ksurl Apr 11, 2023

Choose a reason for hiding this comment

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

Looks like it only pushes on tags rather than every commit?

Copy link
Copy Markdown
Member

@gazpachoking gazpachoking Apr 11, 2023

Choose a reason for hiding this comment

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

It got changed from being in the 'if' block to down here. Should have put my comment on the range so that the diff showed up better. Just curious the motivation.
image

Copy link
Copy Markdown
Contributor Author

@trim21 trim21 Apr 11, 2023 •
edited
Loading

Choose a reason for hiding this comment

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

this step also build the docker image

ci in pr won't build docker image with previous if field， dockerfile is not tested in the pr

Copy link
Copy Markdown
Contributor Author

@trim21 trim21 Apr 11, 2023 •
edited
Loading

Choose a reason for hiding this comment

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

it doesn't actually change anything， previous ci only push and build on the condition (github.event_name == 'push' && github.ref != 'refs/heads/master')， now it build in any condition（if ci triggered）， and push in the condition.

This is what I mean by looks like dockerfile is not actually tested

just like this workflow run: https://github.com/Flexget/Flexget/actions/runs/4659863819/jobs/8247244212

Copy link
Copy Markdown
Member

@gazpachoking gazpachoking Apr 11, 2023

Choose a reason for hiding this comment

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

Ahh, I get it now. Sounds good!

@gazpachoking gazpachoking merged commit 5b75b46 into Flexget:develop Apr 11, 2023
@trim21 trim21 deleted the patch-1 branch April 11, 2023 13:45
stevezau added a commit that referenced this pull request May 24, 2023
@patrickegli @renovate
@stevezau @gazpachoking @pre-commit-ci @FlexGet-Bot @StefftheEmperor2 @trim21 @aidan- @paranoidi @ProteinPig @kingamajick @eSoares @appzer @ianstalk @maxcanna @mnixry @spgomez
Radarr sonarr api v3 - Fix merge conflicts with develop (#3775)
7d510a4 
* Update docker/build-push-action action to v4

* Update irc to download tracker config correctly (#3678)

[fix] irc: Update irc to download tracker config correctly

* Bump isort version in pre-commit

* Close task requests session after execution.
Hopefully helps address issue with file handles being left open.

* [pre-commit.ci] pre-commit autoupdate

updates:
- [github.com/psf/black: 22.12.0 → 23.1.0](psf/black@22.12.0...23.1.0)

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

* v3.5.22

* Prepare v3.5.23.dev

* Fix for empty lazy field

* use better way for cheking for existance of a field

Co-authored-by: Chase Sterling <chase.sterling@gmail.com>

* v3.5.23

* Prepare v3.5.24.dev

* [fix] pyload ng (#3684)

* Fix for pyload ng

---------

Co-authored-by: Stefan Bischoff <stefan.bischoff@power.cloud>

* v3.5.24

* Prepare v3.5.25.dev

* fix

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

* revert space change

* v3.5.25

* Prepare v3.5.26.dev

* [fix] sftp_list host_key accessing exception (#3694)

* v3.5.26

* Prepare v3.5.27.dev

* [add] from_qbittorrent plugin (#3692)

* [change] Add support for 'labels' in Transmission. (#3693)

* v3.5.27

* Prepare v3.5.28.dev

* [fix] memusage debugging module updated for python 3

* [fix] memusage plugin crashes with windows

* [pre-commit.ci] pre-commit autoupdate

updates:
- [github.com/python-poetry/poetry: 1.3.0 → 1.4.0](python-poetry/poetry@1.3.0...1.4.0)

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

* v3.5.28

* Prepare v3.5.29.dev

* [fix] Memory leak when there is no sys.stderr (#3701)

Also clean up the Manager init behavior and logging startup to be
a bit easier to follow.

* v3.5.29

* Prepare v3.5.30.dev

* Add more detailed assertions for entries generated by the sftp_list plugin.

Previously the test cases only focused on if a certain entry existed, not that it contined the correct data. As such this change adds assert_entries() method which allows more detail about the entries generated to be specified as well as checking if any erroneous entires were also generated. This change preceeds the change of symlink handling in the sftp_list plugin to make it easier to assert the URL change in the test.

This change also includes a small bit of cleanup in terms of the task names to make it clearer which variables are been set in that test case.

* Dont normalized every path returned via the sftp_list plugin.

Previously every path returned from the sftp_list plugin was normalised, this meant that symlinks were always resolved before been returned making it impossible to get the symlinks path.

* If a sftp_download entry points to a symlink and delete_origin is true, only remove the symlink, and not it's target.

This changes the behaviour when dealing with entries representing
symlinks. If entry previously represented a symlink it would remove it's
target or in the case of a directory it's contents. This changes that
behaviour such as in the case of a symlink, only the link will be
removed, but not it's target.

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

* Allow newer rpyc versions. fix #3601

* v3.5.30

* Prepare v3.5.31.dev

* [fix] transmission: transmission-rpc deprecated base64 torrent content (#3715)

* fix: transmission-rpc deprecated base64 torrent content

* use patblib.Path

* use shared timeout

* read bytes

* [add] Torznab: Add timeout parameter (#2738)

* Torznab: Add timeout parameter

This is needed for under some environments with many sources on the
torznab, the answer will take more than 30 seconds.
Let the user configure this to adapt according to their needs.

* v3.5.31

* Prepare v3.5.32.dev

* [add] sftp plugin supports additional SSH key types (#3714)

* Handle additional SSH Key.

pysftp doesn't handle  Ed25519Key and ECDSA key types, this change monkeypatches the _set_authentication method in pysftp to add handling for these additional key types.

* v3.5.32

* Prepare v3.5.33.dev

* Pushsafer Notification Service (#3718)

[change] pushsafer: Adds more options to customize notification

* v3.5.33

* Prepare v3.5.34.dev

* Update actions/stale action to v8 (#3725)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* v3.5.34

* Prepare v3.5.35.dev

* [pre-commit.ci] pre-commit autoupdate

updates:
- [github.com/psf/black: 23.1.0 → 23.3.0](psf/black@23.1.0...23.3.0)

* v3.5.35

* Prepare v3.5.36.dev

* v3.5.36

* Prepare v3.5.37.dev

* upgrade to transmission-rpc v4 (#3716)

transmission: Update to use transmission-rpc >=4.0

* v3.6.0

* Prepare v3.6.1.dev

* upgrade to new python syntax (#3738)

Adds a pre-commit hook which updates syntax to cleaner versions available
in newer python versions.

* [change] rarbg: allow using domain_delay and increase default delay. fix #3705
Increase rarbg delay from 3s to 6s.
Allow using domain_delay to override the built in delay for rarbg.
Add an option to not replace existing domain limiter on a session.

* Add ruff to dev dependencies

* Ruff fix E713. not X in Y -> X not in Y

* Ruff fix F541. f-strings without placeholders

* Ruff fix F401. unused imports

* Ruff fix F841. unused variables

* Ruff fix E731. lambda assignment

* Ruff fix F402. shadowed variables

* Ruff fix F821. undefined names

* Add some noqa for purposefully delayed imports

* Ruff fix F811. redefinition of unused names

* Ruff fix E722. bare excepts

* Ruff fix E741. ambiguous variable names

* Fix aria2 config schema I just broke

* Fix sftp plugin?

* add qb package to Docker (#3737)

Move extra docker requirements to pyproject.toml
Build docker image on PRs

* v3.6.1

* Prepare v3.6.2.dev

* Enable ruff in pre-commit.
Enable ruff to do import sorting.
Sort all imports.

* Switch from pyupgrade to ruff in pre-commit.
More pyupgrade type fixes.

* Remove isort and pylint deps.

* Enable pylint error checking on ruff.

* Some fixes for flake8 bugbear warnings.

* Enable flake8 comprehension fixes (C4)
Fix them.

* Enable some more ruff checks.

* from_transmission: Properly set url field to prevent crash. fix #3741

* Simplify by using pathlib instead of os.path

* Enable implicit string concat linting

* Tweak docker webui install for easier maintenance (#3739)

* Update docker.io/python Docker tag to v3.11

* Switch Dockerfile to use dev_tools to bundle webui

---------

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* v3.6.2

* Prepare v3.6.3.dev

* v3.6.3

* Prepare v3.6.4.dev

* [pre-commit.ci] pre-commit autoupdate

updates:
- [github.com/charliermarsh/ruff-pre-commit: v0.0.261 → v0.0.262](astral-sh/ruff-pre-commit@v0.0.261...v0.0.262)

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

* v3.6.4

* Prepare v3.6.5.dev

* Cross build docker images for more arch

* Add rust compiler

This is needed to build cryptography

* Add cargo package manager

* Set cargo to use of the git executable to fetch,

Ref. rust-lang/cargo#10230

* Add git

We need it to allow cargo to fetch packages

* add

* [fix] from_transmission: Don't crash when generating done_date. fix #3745

* fix(deps): update dependency flask to v2.3.2 [security]

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

* Maybe fix renovate/pre-commit wars on vulnerability PRs

* Different try to disable requirements.txt detection for vulnerability prs

* [pre-commit.ci] pre-commit autoupdate

updates:
- [github.com/charliermarsh/ruff-pre-commit: v0.0.262 → v0.0.263](astral-sh/ruff-pre-commit@v0.0.262...v0.0.263)

* v3.6.5

* Prepare v3.6.6.dev

* fix(deps): update dependency flask to v2.2.5 [security]

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

* v3.6.6

* Prepare v3.6.7.dev

* [Add] template filter to parse human-readable file size to bytes (#3750)

* Add template filter to parse human-readable file size to bytes

* pass test

* handle undefined variable case

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

* add si carry support and remove regex anchor

minor improve of regex false-positive match

* tighten up regexp and adjust variable name

* add specified test for `parse_size`

* add case-insensitive matching support

---------

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>

* add provenance: false to docker buildx (#3759)

* Strip timezone from transmission_date_done field to be consistent with other datetimes

* v3.6.7

* Prepare v3.6.8.dev

* [change] Update dockerfile for faster builds and smaller updates

* Optimize docker layers so that dependencies don't get rebuilt when they haven't changed
* CI now caches docker image layers for faster builds

* v3.6.8

* Prepare v3.6.9.dev

* chore(deps): update actions/cache action to v3

* Remove unused docker caching (#3763)

The docker/build-push-action has built in caching. Looks like the explicit cache action wasn't doing anything.

* [fix] download: Fix streaming downloads to disk instead of loading entirely in memory. fix #3762 (#3764)

* Cache intermediate layers for docker builds

* [pre-commit.ci] pre-commit autoupdate (#3765)

updates:
- [github.com/charliermarsh/ruff-pre-commit: v0.0.263 → v0.0.265](astral-sh/ruff-pre-commit@v0.0.263...v0.0.265)

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>

* v3.6.9

* Prepare v3.6.10.dev

* [change] morethantv: Update categories to match website

* Update ubuntu Docker tag to v22 (#3662)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* v3.7.0

* Prepare v3.7.1.dev

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

---------

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: Steve Adams <stevezau@gmail.com>
Co-authored-by: Chase Sterling <chase.sterling@gmail.com>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: FlexGet-Bot <chase.sterling+fg@gmail.com>
Co-authored-by: Stefan Bischoff <stefan.bischoff@power.cloud>
Co-authored-by: StefftheEmperor2 <55734627+StefftheEmperor2@users.noreply.github.com>
Co-authored-by: Trim21 <trim21.me@gmail.com>
Co-authored-by: Aidan Rowe <aidanrowe@gmail.com>
Co-authored-by: paranoidi <marko.koivusalo@gmail.com>
Co-authored-by: ProteinPig <ProteinPig@Gmail.com>
Co-authored-by: kingamjick <kingamajick@gmail.com>
Co-authored-by: eSoares <eSoares@users.noreply.github.com>
Co-authored-by: Kevin Siml <appzer@users.noreply.github.com>
Co-authored-by: ianstalk <ijeaston@gmail.com>
Co-authored-by: Massimiliano Cannarozzo <maxcanna@gmail.com>
Co-authored-by: Johnny Hsieh <32300164+mnixry@users.noreply.github.com>
Co-authored-by: Steve <38679608+spgomez@users.noreply.github.com>
Co-authored-by: Patrick Egli <paedy@P-DESKTOP>
stevezau added a commit that referenced this pull request May 25, 2023
@stevezau @cvium
@pre-commit-ci @patrickegli @renovate @gazpachoking @FlexGet-Bot @StefftheEmperor2 @trim21 @aidan- @paranoidi @ProteinPig @kingamajick @eSoares @appzer @ianstalk @maxcanna @mnixry @spgomez
Update Sonarr/Radarr to use v3 api (#3051)
0e67559 
* [fix] imdb_lookup: updated the scraper for the newish html structure

* fix some tests

* cassettes

* ...

* move to https as imdb now redirects to https by default

* fix test_imdb tests

* fix imdb parser tests

* fix imdb tests

* more updates to vcr recordings to fix imdb tests

* use api v3

* fix profile endpoint

* fix profileid name

* language id required

* fixed

* Initial rewrite

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

* Initial rewrite

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

* fix test cassettes

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

* fix exact match

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

* fix test imdb cache

* fix nfo tests

* refactor exact testing

* fix text exists

* Fix test

* Radarr sonarr api v3 - Fix merge conflicts (#3771)

* Revert unwanted changes

Restore api.py and test_metainfo.py to develop version.
Delete imdb.py (not existent on develop)

tmp

* Adapt cassettes for radarr/sonarr tests

---------

Co-authored-by: Patrick Egli <paedy@P-DESKTOP>

* Radarr sonarr api v3 - Fix merge conflicts with develop (#3775)

* Update docker/build-push-action action to v4

* Update irc to download tracker config correctly (#3678)

[fix] irc: Update irc to download tracker config correctly

* Bump isort version in pre-commit

* Close task requests session after execution.
Hopefully helps address issue with file handles being left open.

* [pre-commit.ci] pre-commit autoupdate

updates:
- [github.com/psf/black: 22.12.0 → 23.1.0](psf/black@22.12.0...23.1.0)

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

* v3.5.22

* Prepare v3.5.23.dev

* Fix for empty lazy field

* use better way for cheking for existance of a field

Co-authored-by: Chase Sterling <chase.sterling@gmail.com>

* v3.5.23

* Prepare v3.5.24.dev

* [fix] pyload ng (#3684)

* Fix for pyload ng

---------

Co-authored-by: Stefan Bischoff <stefan.bischoff@power.cloud>

* v3.5.24

* Prepare v3.5.25.dev

* fix

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

* revert space change

* v3.5.25

* Prepare v3.5.26.dev

* [fix] sftp_list host_key accessing exception (#3694)

* v3.5.26

* Prepare v3.5.27.dev

* [add] from_qbittorrent plugin (#3692)

* [change] Add support for 'labels' in Transmission. (#3693)

* v3.5.27

* Prepare v3.5.28.dev

* [fix] memusage debugging module updated for python 3

* [fix] memusage plugin crashes with windows

* [pre-commit.ci] pre-commit autoupdate

updates:
- [github.com/python-poetry/poetry: 1.3.0 → 1.4.0](python-poetry/poetry@1.3.0...1.4.0)

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

* v3.5.28

* Prepare v3.5.29.dev

* [fix] Memory leak when there is no sys.stderr (#3701)

Also clean up the Manager init behavior and logging startup to be
a bit easier to follow.

* v3.5.29

* Prepare v3.5.30.dev

* Add more detailed assertions for entries generated by the sftp_list plugin.

Previously the test cases only focused on if a certain entry existed, not that it contined the correct data. As such this change adds assert_entries() method which allows more detail about the entries generated to be specified as well as checking if any erroneous entires were also generated. This change preceeds the change of symlink handling in the sftp_list plugin to make it easier to assert the URL change in the test.

This change also includes a small bit of cleanup in terms of the task names to make it clearer which variables are been set in that test case.

* Dont normalized every path returned via the sftp_list plugin.

Previously every path returned from the sftp_list plugin was normalised, this meant that symlinks were always resolved before been returned making it impossible to get the symlinks path.

* If a sftp_download entry points to a symlink and delete_origin is true, only remove the symlink, and not it's target.

This changes the behaviour when dealing with entries representing
symlinks. If entry previously represented a symlink it would remove it's
target or in the case of a directory it's contents. This changes that
behaviour such as in the case of a symlink, only the link will be
removed, but not it's target.

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

* Allow newer rpyc versions. fix #3601

* v3.5.30

* Prepare v3.5.31.dev

* [fix] transmission: transmission-rpc deprecated base64 torrent content (#3715)

* fix: transmission-rpc deprecated base64 torrent content

* use patblib.Path

* use shared timeout

* read bytes

* [add] Torznab: Add timeout parameter (#2738)

* Torznab: Add timeout parameter

This is needed for under some environments with many sources on the
torznab, the answer will take more than 30 seconds.
Let the user configure this to adapt according to their needs.

* v3.5.31

* Prepare v3.5.32.dev

* [add] sftp plugin supports additional SSH key types (#3714)

* Handle additional SSH Key.

pysftp doesn't handle  Ed25519Key and ECDSA key types, this change monkeypatches the _set_authentication method in pysftp to add handling for these additional key types.

* v3.5.32

* Prepare v3.5.33.dev

* Pushsafer Notification Service (#3718)

[change] pushsafer: Adds more options to customize notification

* v3.5.33

* Prepare v3.5.34.dev

* Update actions/stale action to v8 (#3725)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* v3.5.34

* Prepare v3.5.35.dev

* [pre-commit.ci] pre-commit autoupdate

updates:
- [github.com/psf/black: 23.1.0 → 23.3.0](psf/black@23.1.0...23.3.0)

* v3.5.35

* Prepare v3.5.36.dev

* v3.5.36

* Prepare v3.5.37.dev

* upgrade to transmission-rpc v4 (#3716)

transmission: Update to use transmission-rpc >=4.0

* v3.6.0

* Prepare v3.6.1.dev

* upgrade to new python syntax (#3738)

Adds a pre-commit hook which updates syntax to cleaner versions available
in newer python versions.

* [change] rarbg: allow using domain_delay and increase default delay. fix #3705
Increase rarbg delay from 3s to 6s.
Allow using domain_delay to override the built in delay for rarbg.
Add an option to not replace existing domain limiter on a session.

* Add ruff to dev dependencies

* Ruff fix E713. not X in Y -> X not in Y

* Ruff fix F541. f-strings without placeholders

* Ruff fix F401. unused imports

* Ruff fix F841. unused variables

* Ruff fix E731. lambda assignment

* Ruff fix F402. shadowed variables

* Ruff fix F821. undefined names

* Add some noqa for purposefully delayed imports

* Ruff fix F811. redefinition of unused names

* Ruff fix E722. bare excepts

* Ruff fix E741. ambiguous variable names

* Fix aria2 config schema I just broke

* Fix sftp plugin?

* add qb package to Docker (#3737)

Move extra docker requirements to pyproject.toml
Build docker image on PRs

* v3.6.1

* Prepare v3.6.2.dev

* Enable ruff in pre-commit.
Enable ruff to do import sorting.
Sort all imports.

* Switch from pyupgrade to ruff in pre-commit.
More pyupgrade type fixes.

* Remove isort and pylint deps.

* Enable pylint error checking on ruff.

* Some fixes for flake8 bugbear warnings.

* Enable flake8 comprehension fixes (C4)
Fix them.

* Enable some more ruff checks.

* from_transmission: Properly set url field to prevent crash. fix #3741

* Simplify by using pathlib instead of os.path

* Enable implicit string concat linting

* Tweak docker webui install for easier maintenance (#3739)

* Update docker.io/python Docker tag to v3.11

* Switch Dockerfile to use dev_tools to bundle webui

---------

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* v3.6.2

* Prepare v3.6.3.dev

* v3.6.3

* Prepare v3.6.4.dev

* [pre-commit.ci] pre-commit autoupdate

updates:
- [github.com/charliermarsh/ruff-pre-commit: v0.0.261 → v0.0.262](astral-sh/ruff-pre-commit@v0.0.261...v0.0.262)

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

* v3.6.4

* Prepare v3.6.5.dev

* Cross build docker images for more arch

* Add rust compiler

This is needed to build cryptography

* Add cargo package manager

* Set cargo to use of the git executable to fetch,

Ref. rust-lang/cargo#10230

* Add git

We need it to allow cargo to fetch packages

* add

* [fix] from_transmission: Don't crash when generating done_date. fix #3745

* fix(deps): update dependency flask to v2.3.2 [security]

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

* Maybe fix renovate/pre-commit wars on vulnerability PRs

* Different try to disable requirements.txt detection for vulnerability prs

* [pre-commit.ci] pre-commit autoupdate

updates:
- [github.com/charliermarsh/ruff-pre-commit: v0.0.262 → v0.0.263](astral-sh/ruff-pre-commit@v0.0.262...v0.0.263)

* v3.6.5

* Prepare v3.6.6.dev

* fix(deps): update dependency flask to v2.2.5 [security]

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

* v3.6.6

* Prepare v3.6.7.dev

* [Add] template filter to parse human-readable file size to bytes (#3750)

* Add template filter to parse human-readable file size to bytes

* pass test

* handle undefined variable case

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

* add si carry support and remove regex anchor

minor improve of regex false-positive match

* tighten up regexp and adjust variable name

* add specified test for `parse_size`

* add case-insensitive matching support

---------

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>

* add provenance: false to docker buildx (#3759)

* Strip timezone from transmission_date_done field to be consistent with other datetimes

* v3.6.7

* Prepare v3.6.8.dev

* [change] Update dockerfile for faster builds and smaller updates

* Optimize docker layers so that dependencies don't get rebuilt when they haven't changed
* CI now caches docker image layers for faster builds

* v3.6.8

* Prepare v3.6.9.dev

* chore(deps): update actions/cache action to v3

* Remove unused docker caching (#3763)

The docker/build-push-action has built in caching. Looks like the explicit cache action wasn't doing anything.

* [fix] download: Fix streaming downloads to disk instead of loading entirely in memory. fix #3762 (#3764)

* Cache intermediate layers for docker builds

* [pre-commit.ci] pre-commit autoupdate (#3765)

updates:
- [github.com/charliermarsh/ruff-pre-commit: v0.0.263 → v0.0.265](astral-sh/ruff-pre-commit@v0.0.263...v0.0.265)

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>

* v3.6.9

* Prepare v3.6.10.dev

* [change] morethantv: Update categories to match website

* Update ubuntu Docker tag to v22 (#3662)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* v3.7.0

* Prepare v3.7.1.dev

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

---------

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: Steve Adams <stevezau@gmail.com>
Co-authored-by: Chase Sterling <chase.sterling@gmail.com>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: FlexGet-Bot <chase.sterling+fg@gmail.com>
Co-authored-by: Stefan Bischoff <stefan.bischoff@power.cloud>
Co-authored-by: StefftheEmperor2 <55734627+StefftheEmperor2@users.noreply.github.com>
Co-authored-by: Trim21 <trim21.me@gmail.com>
Co-authored-by: Aidan Rowe <aidanrowe@gmail.com>
Co-authored-by: paranoidi <marko.koivusalo@gmail.com>
Co-authored-by: ProteinPig <ProteinPig@Gmail.com>
Co-authored-by: kingamjick <kingamajick@gmail.com>
Co-authored-by: eSoares <eSoares@users.noreply.github.com>
Co-authored-by: Kevin Siml <appzer@users.noreply.github.com>
Co-authored-by: ianstalk <ijeaston@gmail.com>
Co-authored-by: Massimiliano Cannarozzo <maxcanna@gmail.com>
Co-authored-by: Johnny Hsieh <32300164+mnixry@users.noreply.github.com>
Co-authored-by: Steve <38679608+spgomez@users.noreply.github.com>
Co-authored-by: Patrick Egli <paedy@P-DESKTOP>

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

---------

Co-authored-by: cvium <clausvium@gmail.com>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: patrickegli <patrickegli@users.noreply.github.com>
Co-authored-by: Patrick Egli <paedy@P-DESKTOP>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: Chase Sterling <chase.sterling@gmail.com>
Co-authored-by: FlexGet-Bot <chase.sterling+fg@gmail.com>
Co-authored-by: Stefan Bischoff <stefan.bischoff@power.cloud>
Co-authored-by: StefftheEmperor2 <55734627+StefftheEmperor2@users.noreply.github.com>
Co-authored-by: Trim21 <trim21.me@gmail.com>
Co-authored-by: Aidan Rowe <aidanrowe@gmail.com>
Co-authored-by: paranoidi <marko.koivusalo@gmail.com>
Co-authored-by: ProteinPig <ProteinPig@Gmail.com>
Co-authored-by: kingamjick <kingamajick@gmail.com>
Co-authored-by: eSoares <eSoares@users.noreply.github.com>
Co-authored-by: Kevin Siml <appzer@users.noreply.github.com>
Co-authored-by: ianstalk <ijeaston@gmail.com>
Co-authored-by: Massimiliano Cannarozzo <maxcanna@gmail.com>
Co-authored-by: Johnny Hsieh <32300164+mnixry@users.noreply.github.com>
Co-authored-by: Steve <38679608+spgomez@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@gazpachoking gazpachoking gazpachoking left review comments

+1 more reviewer

@ksurl ksurl ksurl left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@trim21 @gazpachoking @ksurl