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.

Sign up for GitHub

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

Jump to bottom

docker swarm 20.10.12 | "docker service update" requires exactly 1 argument. #83

Closed
Takalele opened this issue Nov 12, 2022 · 2 comments
Closed

docker swarm 20.10.12 | "docker service update" requires exactly 1 argument. #83

Takalele opened this issue Nov 12, 2022 · 2 comments

Comments

@Takalele
Copy link

Hi,

shepherd seams not to be working with swarm anymore, any idea?

house_keeping_service_shepherd.1.b3rmqmqvd2zh@docker-swarm    | Sat Nov 12 19:51:14 CET 2022 Trying to update service apps_service_pdns_dns_admin with image ngoduykhanh/powerdns-admin:latest
house_keeping_service_shepherd.1.b3rmqmqvd2zh@docker-swarm    | "docker service update" requires exactly 1 argument.
house_keeping_service_shepherd.1.b3rmqmqvd2zh@docker-swarm    | See 'docker service update --help'.
house_keeping_service_shepherd.1.b3rmqmqvd2zh@docker-swarm    |
house_keeping_service_shepherd.1.b3rmqmqvd2zh@docker-swarm    | Usage:  docker service update [OPTIONS] SERVICE
house_keeping_service_shepherd.1.b3rmqmqvd2zh@docker-swarm    |
house_keeping_service_shepherd.1.b3rmqmqvd2zh@docker-swarm    | Update a service
house_keeping_service_shepherd.1.b3rmqmqvd2zh@docker-swarm    | Sat Nov 12 19:51:14 CET 2022 Service apps_service_pdns_dns_admin update failed on shepherd!
house_keeping_service_shepherd.1.b3rmqmqvd2zh@docker-swarm    | Sat Nov 12 19:51:14 CET 2022 Rolling apps_service_pdns_dns_admin back

docker version 20.10.12 running in swarm mode

BR Takalele
@etzelc
Copy link

etzelc commented Dec 2, 2022

How did you specify your env variables?
No quotes are used in a docker compose script:

  shepherd:
    image: mazzolino/shepherd:latest
    volumes:
      - /var/run/docker.sock:/var/run/docker.sock
    environment: 
      - UPDATE_OPTIONS=--force

Here you find a larger example: 6829d25

@Takalele
Copy link
Author

Takalele commented Dec 3, 2022

Hi @etzelc,

thank you - thats on me.

BR
Takalele

@Takalele Takalele closed this as completed Dec 3, 2022
moschlar added a commit that referenced this issue Oct 30, 2023
@moschlar
Release new major version 1.8.0
01d070a 
[Full Changelog](0.7.0...1.8.0)

**Breaking changes:**

- The docker image registry location has been changed to the containrrr organisation:
  `containrrr/shepherd`

**Implemented enhancements:**

- armhf support [\#108](#108)
- Switch to official docker image v24 [\#107](#107) ([djmaze](https://github.com/djmaze))
- Restrict runtime of "docker service update" using "timeout" [\#98](#98) ([fooflington](https://github.com/fooflington))
- Add example for usage with swarm-cronjob [\#89](#89) ([djmaze](https://github.com/djmaze))

**Fixed bugs:**

- Can't update some services: no such manifest [\#105](#105)
- Service gets stuck when calling "docker service update" and won't progress [\#97](#97)
- fix: docker service update with `--detach=false` hangs on services wi… [\#104](#104) ([AliRezaBeitari](https://github.com/AliRezaBeitari))
- Fix defunc VERBOSE handling [\#91](#91) ([sebthom](https://github.com/sebthom))

**Closed issues:**

- How does it determine if there is an update or not? [\#111](#111)
- Looking for a new maintainer [\#96](#96)
- New OCI manifest issue [\#92](#92)
- Run service update at a fixed time [\#88](#88)
- docker swarm 20.10.12 | "docker service update" requires exactly 1 argument. [\#83](#83)
- Error updating service, does not exist or it is not available when using a duplicate registry [\#78](#78)

**Merged pull requests:**

- Add apprise type and additional error notification [\#118](#118) ([andyloree](https://github.com/andyloree))
- Rename image in docs [\#114](#114) ([moschlar](https://github.com/moschlar))
- Fix release workflow [\#113](#113) ([moschlar](https://github.com/moschlar))
- Update README.md [\#103](#103) ([andrew-dixon](https://github.com/andrew-dixon))
- ci: add basic github actions for build/release [\#101](#101) ([piksel](https://github.com/piksel))
- correct misleading description of WITH\_NO\_RESOLVE\_IMAGE [\#100](#100) ([alex-vg](https://github.com/alex-vg))
- Move example configs to their own folder [\#99](#99) ([djmaze](https://github.com/djmaze))
- Add documentation about `REGISTRIES_FILE` [\#94](#94) ([tito](https://github.com/tito))
- Minor refactoring [\#90](#90) ([sebthom](https://github.com/sebthom))

Closes: #117
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@etzelc @Takalele