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

Feature Request - Command-line param to process only a specific collection #26

Closed
maciejzgadzaj opened this issue Feb 12, 2021 · 13 comments · Fixed by #50
Closed

Feature Request - Command-line param to process only a specific collection #26

maciejzgadzaj opened this issue Feb 12, 2021 · 13 comments · Fixed by #50
Assignees
Labels
enhancement New feature or request

Comments

@maciejzgadzaj
Copy link
Contributor

I have the script set up in a docker container, and it runs at 3 am as planned.

But testing different collection configurations (specifically external lists) I am also running it manually and checking the results.

Would be great if I could provide an extra command line param to make it reprocess a specific collection only, instead of all of them each time.

And yes, I know I can just leave only the collection config I am testing in my Movies.yml file, but still... just an idea. :)

@meisnate12
Copy link
Member

i think i can come up with something

@meisnate12 meisnate12 self-assigned this Feb 13, 2021
@meisnate12 meisnate12 added the enhancement New feature or request label Feb 13, 2021
@meisnate12 meisnate12 changed the title Command-line param to process only a specific collection Feature Request - Command-line param to process only a specific collection Feb 13, 2021
meisnate12 added a commit that referenced this issue Feb 15, 2021
@meisnate12
Copy link
Member

ok the develop branch has this feature can you test? run test using -rt, --test, --run-test. Add test: true to any metadata or collection object to have them run in test mode. I also turned on Debugging when running test mode so there will be more output.

@maciejzgadzaj
Copy link
Contributor Author

maciejzgadzaj commented Feb 15, 2021

Pulled, rebuilt, can confirm that it works as described (so far at least) - thank you very much!

Not quite what I've originally imagined, was thinking more of sth like python3 plex_meta_manager.py --run --collection=Comedy,Crime (without having to additionally modify the Movies.yml file for this) - but hey, it's your app after all - as long as it works I'm happy. :)

@meisnate12
Copy link
Member

I choose this way because like you I run multiple docker containers, one to run the schedule and one with the -r option turned on. So I wanted to be able to spin up another that I can run tests from without having to recreate the container each time. this way I can just run the test container

@meisnate12
Copy link
Member

also leaving open untill pushed to master

@meisnate12 meisnate12 reopened this Feb 15, 2021
@maciejzgadzaj
Copy link
Contributor Author

So I wanted to be able to spin up another that I can run tests from without having to recreate the container each time. this way I can just run the test container

Not sure exactly how exactly are you using all your containers, but I might have a solution which you might like for this. Coming to you later tonight, if I have time to polish it locally first.

@meisnate12
Copy link
Member

I'm excited to see what you've got!

@maciejzgadzaj
Copy link
Contributor Author

#34

maciejzgadzaj added a commit to maciejzgadzaj/Plex-Meta-Manager that referenced this issue Feb 15, 2021
@maciejzgadzaj
Copy link
Contributor Author

Also had some fun and came up with the above MR... 🤷

meisnate12 added a commit that referenced this issue Feb 16, 2021
…ion-param

#26 command-line param to process only a specific collection(s)
@maciejzgadzaj
Copy link
Contributor Author

And then again - we're talking *nixes here only. What about Windows? My guess would be it wouldn't work there either, so it's already not 100% generic. Personally I think I'd add it - considering that it's mostly alpine and buster on dockerhub - and see if anyone complains. :D (If it was my repo obviously.)

@meisnate12 meisnate12 mentioned this issue Feb 20, 2021
@Tharic99
Copy link

Did this break the --run command by itself?

python plex_meta_manager.py --run

Returns the following:

|====================================================================================================|
|                                                                                                    |
|        ____  _             __  __      _          __  __                                           |
|       |  _ \| | _____  __ |  \/  | ___| |_ __ _  |  \/  | __ _ _ __   __ _  __ _  ___ _ __         |
|       | |_) | |/ _ \ \/ / | |\/| |/ _ \ __/ _` | | |\/| |/ _` | '_ \ / _` |/ _` |/ _ \ '__|        |
|       |  __/| |  __/>  <  | |  | |  __/ || (_| | | |  | | (_| | | | | (_| | (_| |  __/ |           |
|       |_|   |_|\___/_/\_\ |_|  |_|\___|\__\__,_| |_|  |_|\__,_|_| |_|\__,_|\__, |\___|_|           |
|                                                                            |___/                   |
|           Version: 1.2.2                                                                           |
|====================================================================================================|
|====================================================================================================|
|                                            Starting Run                                            |
|====================================================================================================|
| Locating config...                                                                                 |
| Using //config/config.yml as config                                                                |
|====================================================================================================|
| Using cache database at //config/config.cache                                                      |
|====================================================================================================|
| Connecting to TMDb...                                                                              |
| TMDb Connection Successful                                                                         |
|====================================================================================================|
| Connecting to Trakt...                                                                             |
| Trakt Connection Successful                                                                        |
|====================================================================================================|
| Connecting to My Anime List...                                                                     |
| Config Error: mal sub-attribute client_id is blank                                                 |
| My Anime List Connection Failed                                                                    |
|====================================================================================================|
| Connecting to Plex Libraries...                                                                    |
|====================================================================================================|
| Connecting to Movies Library...                                                                    |
| Using Asset Directory: //config/assets                                                             |
| Movies Library Connection Successful                                                               |
| Connecting to Movies library's Radarr...                                                           |
| Movies library's Radarr Connection Successful                                                      |
| Connecting to Movies library's Sonarr...                                                           |
| Movies library's Sonarr Connection Successful                                                      |
| Connecting to Movies library's Tautulli...                                                         |
| Movies library's Tautulli Connection Successful                                                    |
|====================================================================================================|
| Connecting to TV Shows Library...                                                                  |
| 'metadata_path'                                                                                    |
|                                                                                                    |
|====================================================================================================|
|                                            Finished Run                                            |
|====================================================================================================|
# 

But it doesn't actually "do" anything...

@meisnate12
Copy link
Member

No that's a different error if you could post a new issue with your meta.log file?

@Tharic99
Copy link

No that's a different error if you could post a new issue with your meta.log file?

I figured it out. Since it moved stuff around in my config with the upgraded version over the weekends, I did not have a TV Shows.yml config file created yet and that was causing this error. Issue #55 opened for this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants