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

1.8.0 testing crunchy watch panics on docker #428

Closed
altondinsmore opened this issue Feb 12, 2018 · 4 comments
Closed

1.8.0 testing crunchy watch panics on docker #428

altondinsmore opened this issue Feb 12, 2018 · 4 comments
Assignees
Milestone

Comments

@altondinsmore
Copy link

Chrunchy Watch 1.8.0 was not found so modified the run.sh and changed last line to
-d $CCP_IMAGE_PREFIX/crunchy-watch:centos7-10.1-1.7.1
When running run.sh it looks good but issuing docker logs watch reviles it failed with a panic.

docker logs watch
WARNING: PG_PRIMARY_SERVICE is deprecated and will be removed in a future release, use CRUNCHY_WATCH_PRIMARY instead.
WARNING: PG_PRIMARY_PORT is deprecated and will be removed in a future release, use CRUNCHY_WATCH_PRIMARY_PORT instead.
WARNING: PG_REPLICA_SERVICE is deprecated and will be removed in a future release, use CRUNCHY_WATCH_REPLICA instead.
WARNING: PG_PRIMARY_USER is deprecated and will be removed in a future release, use CRUNCHY_WATCH_USERNAME instead.
WARNING: PG_DATABASE is deprecated and will be removed in a future release, use CRUNCHY_WATCH_DATABASE instead.
WARNING: WATCH_PRE_HOOK is deprecated and will be removed in a future release, use CRUNCHY_WATCH_PRE_HOOK instead.
WARNING: WATCH_POST_HOOK is deprecated and will be removed in a future release, use CRUNCHY_WATCH_POST_HOOK instead.
WARNING: SLEEP_TIME is deprecated and will be removed in a future release, use CRUNCHY_WATCH_HEALTHCHECK_INTERVAL instead.
Starting crunchy watch
Started...
panic: runtime error: index out of range
goroutine 1 [running]:
main.main()
/opt/cdev/src/github.com/crunchydata/crunchy-watch/main.go:170 +0x1497

@jmccormick2001
Copy link
Contributor

@davecramer please verify this is still a bug

@davecramer
Copy link
Contributor

Yes, it is still a bug

@davecramer
Copy link
Contributor

Much more than just this see CrunchyData/crunchy-watch#43 for real issues

@davecramer
Copy link
Contributor

fixed in CrunchyData/crunchy-watch#47

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

No branches or pull requests

3 participants