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

fix: Vector fails to start container due to --watch-config flag #5197

Merged
merged 1 commit into from
May 25, 2022

Conversation

holamendi
Copy link
Contributor

Vector doesn't accept an argument after the --watch-config flag anymore, so executing dokku logs:vector-start fails to start the container.

The output of dokku logs:vector-logs:

-----> Vector container logs
 !     USAGE:
 !         vector [OPTIONS] [SUBCOMMAND]
 !     For more information try --help
 !     error: Found argument '1' which wasn't expected, or isn't valid in this context

The output of docker ps --no-trunc:

CONTAINER ID                                                       IMAGE                           COMMAND                                                                              CREATED          STATUS                          PORTS      NAMES
f3bb3fae0464081c3c8c39cfb7f777ed920a8bd4f89392bc922d949e9e721886   timberio/vector:latest-debian   "/usr/bin/vector --config /etc/vector/vector.json --watch-config 1"                  35 minutes ago   Restarting (2) 30 seconds ago              vector

@holamendi holamendi changed the title Fix: Vector fails to start container due to --watch-config flag fix: Vector fails to start container due to --watch-config flag May 25, 2022
@josegonzalez josegonzalez merged commit d30fb7b into dokku:master May 25, 2022
@holamendi
Copy link
Contributor Author

I see the CI tests failed on master: https://github.com/dokku/dokku/runs/6595025431?check_suite_focus=true#step:5:41

Could it be due to a Vector version difference?

@josegonzalez
Copy link
Member

Yeah seems like maybe you were using a newer version manually? I'm going to revert this for now and we can merge this in later once I do the minor release (next release is a patch).

@holamendi
Copy link
Contributor Author

You're right. I used dokku logs:vector-start --vector-image timberio/vector:latest-debian. Sorry for the confusion.

Dokku's Vector doc page shows that same command as an example for starting the container with a custom Image, which in this case broke the plugin.

@josegonzalez
Copy link
Member

Well we start it with an image that is tested and compatible. If the upstream image changes, it changes, can't do much about that.

@holamendi
Copy link
Contributor Author

Totally, sorry again for the false alarm.

@holamendi holamendi deleted the fix-vector-start-container branch May 26, 2022 14:47
github-actions bot pushed a commit that referenced this pull request May 29, 2022
# History

## 0.27.5

Install/update via the bootstrap script:

```shell
wget https://raw.githubusercontent.com/dokku/dokku/v0.27.5/bootstrap.sh
sudo DOKKU_TAG=v0.27.5 bash bootstrap.sh
```

### Bug Fixes

- #5183: @josegonzalez Filter apps when verifying app names
- #5199: @josegonzalez Revert "fix: Vector fails to start container due to --watch-config flag"
- #5197: @holamendi fix: Vector fails to start container due to --watch-config flag
- #5192: @josegonzalez Confirm installation of universe packages when installing on ubuntu
- #5193: @josegonzalez Force-set correct permissions on env files
- #5181: @josegonzalez Ensure we default SSH_NAME to NAME when filtering apps

### New Features

- #5196: @josegonzalez Disable herokuish usage on both armhf and arm64 platforms
- #5194: @josegonzalez Upgrade base Dockerfile image to phusion/baseimage:focal-1.2.0

### Other

- #5200: @dependabot[bot] chore(deps): bump rack from 2.2.3 to 2.2.3.1 in /tests/apps/ruby
- #5198: @dependabot[bot] chore(deps): bump grunt from 1.5.2 to 1.5.3 in /tests/apps/multi
- #5187: @dependabot[bot] chore(deps-dev): bump heroku/heroku-buildpack-php from 216 to 217 in /tests/apps/php
josegonzalez added a commit that referenced this pull request Aug 10, 2022
Also drop value for --watch-config flag, as this is now properly parsed as a boolean value.

Refs #5197
@josegonzalez
Copy link
Member

This should be fixed in the next minor :)

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

Successfully merging this pull request may close these issues.

None yet

2 participants