Skip to content

Docker restart/pull handling updates. Minor refactoring.#271

Merged
mag37 merged 2 commits into
mag37:mainfrom
vorezal:feature/docker-handling
May 15, 2026
Merged

Docker restart/pull handling updates. Minor refactoring.#271
mag37 merged 2 commits into
mag37:mainfrom
vorezal:feature/docker-handling

Conversation

@vorezal

@vorezal vorezal commented Mar 12, 2026

Copy link
Copy Markdown
Contributor

This PR:

  • Skips restarting already restarted stacks
  • Skips failed image pulls
  • Moves some stuff around/minor refactoring to allow other bash functions to be used in the print_backups function

This is the first PR in my attempt to separate features in #262. A log feature only PR will follow eventually. This one is intended to address #260.

@mag37

mag37 commented Mar 12, 2026

Copy link
Copy Markdown
Owner

Nice one!
I've just now prepared a release with a few other changes (all in main currently), so I'll hold this merge a little - wont have time to properly run it through my tests until during the weekend.

No rush with the log feature, that might need some discussion and testing etc anyway to reach a good baseline.

@mag37

mag37 commented Apr 30, 2026

Copy link
Copy Markdown
Owner

Sorry @vorezal - other things sprung up in between (as you know) and time has been scarce.

I just pushed another release to have one fresh before merging this.
I'll work out the merge conflicts (dosnt look like much) and do some testing locally, so nothing else affected this change, before I merge it to main. I might look into squeezing in some more cleaning and bump it to v0.8.0 to mark a new chapter with this 😄

@vorezal

vorezal commented Apr 30, 2026

Copy link
Copy Markdown
Contributor Author

Thanks! Sounds good, and no worries. Time is definitely scarce all around.

@mag37 mag37 merged commit 407e9f4 into mag37:main May 15, 2026
@mag37

mag37 commented May 15, 2026

Copy link
Copy Markdown
Owner

Epic work as always @vorezal - the conflicts were minor, should've done this earlier but just needed the headspace to feel I grasped things for testing etc.

Ran it through a few tests with containers in different states and stacks. Thank you again.

@vorezal

vorezal commented May 21, 2026

Copy link
Copy Markdown
Contributor Author

Awesome. Thanks for merging!

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

when one container update fails, all other tasks are stopped/failed/unfinished

2 participants