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

nfsv3driver job ctl script errors aren't sent to log files #6

Closed
youngm opened this issue Mar 27, 2017 · 2 comments
Closed

nfsv3driver job ctl script errors aren't sent to log files #6

youngm opened this issue Mar 27, 2017 · 2 comments
Labels
accepted

Comments

@youngm
Copy link

youngm commented Mar 27, 2017 •
edited
Loading

My nfsv3driver got into a state where it wouldn't start. tailing the logs showed no log messages, the job simply wouldn't start. Running the ctl script manually nfsv3driver_ctl start showed me:

+ RUN_DIR=/var/vcap/sys/run/nfsv3driver
+ LOG_DIR=/var/vcap/sys/log/nfsv3driver
+ PIDFILE=/var/vcap/sys/run/nfsv3driver/nfsv3driver.pid
+ case $1 in
+ id -u cvcap
+ mkdir -p /var/vcap/sys/run/nfsv3driver
+ chown -R cvcap:cvcap /var/vcap/sys/run/nfsv3driver
+ mkdir -p /var/vcap/sys/log/nfsv3driver
+ chown -R cvcap:cvcap /var/vcap/sys/log/nfsv3driver
+ mkdir -p /var/vcap/data/voldrivers
+ chmod 777 /var/vcap/data/voldrivers
+ mkdir -p /var/vcap/data/volumes/nfs
+ chown -R cvcap:cvcap /var/vcap/data/volumes/nfs
chown: cannot access ‘/var/vcap/data/volumes/nfs/2f5673df-3f08-4ae1-98e8-06819a5f6f6b658484ad-867a-4e2e-9c97-3edce02a92dc’: Transport endpoint is not connected

I'm still tracking down how I got into this state and may create another issue for that. But, at the very least I think this is some information that should be appearing in the nfsv3driver logs.

Other cf jobs use a syslog_utils.sh script to tee all console output to files in the /var/vcap/sys/logs directory.

Something to consider.
@cf-gitbot
Copy link

We have created an issue in Pivotal Tracker to manage this:

https://www.pivotaltracker.com/story/show/142551171

The labels on this github issue will be updated when the story is started.

@julian-hj
Copy link
Member

fixed in the master branch.

Kaixiang added a commit that referenced this issue May 15, 2018
@Kaixiang @mariash
bump nfsbroker brokerapi
8668cbe 
Submodule src/code.cloudfoundry.org/nfsbroker 7b95add..0ea62c7:
  > Merge pull request #6 from jmcarp/bump-brokerapi
Submodule src/github.com/pivotal-cf/brokerapi 6d25b93..30e0c68:
  > Add copyright notice to files where it was missing
  > Update NOTICE
  > Undo changes to enforce passing of org and space uids
  > Improve OSBAPI compliance (check missing props and API version header)
  > update gitignore
  > Mention dep in README
  > Force go get to download test dependencies
  > Download dependencies before running tests in travis
  > Remove vendor folder
  > Catalog endpoint can return 500 as status code
  > Do not initialise AdditionalMetadata map if it has no properties
  > Remove hard coding of field names using reflection
  > Simplify alias code during (un)marshaling
  > Merge pull request #51 from carlo-colombo/master
  > Lock gorilla/mux and testflight version
  > Bump dependencies
  > Rename Schema struct field to Parameters
  > Adhere to OSBAPI GET /v2/catalog spec
  > Adhere to OSBAPI PATCH /v2/service_instance/:instance_id/service_bindings spec
  > Adhere to OSBAPI PUT/v2/service_instance/:instance_id/service_bindings/:binding_id spec
  > Test for update context
  > Merge pull request #49 from firasnajjar/patch-1
  > Move deprovision bad request tests to correct location
  > Include log tests for invalid unbind requests
  > Improve error response and logging
  > Adhere to OSB spec for unbind request
  > Link to OSBAPI spec instead of CF docs
  > Loosen X-Broker-API-Version check to require 2.x rather than 2.13
  > adhere to OSBAPI spec
  > Lock lager version
  > Godep is no longer used
  > Replaced godep with dep
  > Merge pull request #47 from camelpunch/ItBecomesSpecify
  > add test case for Bind request GetRawContext and GetRawParameters
  > Merge pull request #45 from liorokman/master
  > BindResource can include credential client ID
  > Introduce `shareable` to ServiceMetadata
  > Make ServiceBroker counterfeitable

Signed-off-by: Maria Shaldibina <mshaldibina@pivotal.io>
julian-hj pushed a commit that referenced this issue Nov 7, 2019
bump nfsv3driver
3c399e7 
Submodule src/code.cloudfoundry.org/nfsv3driver bc0fe75..28eb333:
  > Merge pull request #6 from cloudfoundry/dependabot/go_modules/github.com/onsi/ginkgo-1.10.3
  > Merge pull request #7 from cloudfoundry/dependabot/go_modules/github.com/onsi/gomega-1.7.1
julian-hj added a commit that referenced this issue May 1, 2020
@julian-hj
bump nfsv3driver
fb579d8 
Submodule src/code.cloudfoundry.org/nfsv3driver bc0fe75..8b6921d:
  > Implemented a cache option
  > go mod vendor
  > Merge pull request #22 from cloudfoundry/dependabot/go_modules/code.cloudfoundry.org/volumedriver-0.5.0
  > 🤖 go mod vendor
  > Merge pull request #21 from cloudfoundry/dependabot/go_modules/code.cloudfoundry.org/goshims-0.3.0
  > 🤖 go mod vendor
  > Merge pull request #20 from cloudfoundry/dependabot/go_modules/code.cloudfoundry.org/volumedriver-0.4.0
  > go mod vendor
  > Update go dependencies
  > Remove github actions
  > Merge pull request #19 from cloudfoundry/go-mod-tidy-20200311213050
  > Merge pull request #17 from cloudfoundry/dependabot/go_modules/code.cloudfoundry.org/volumedriver-0.3.0
  > Merge pull request #18 from cloudfoundry/go-mod-tidy-20200311212549
  > Merge pull request #16 from cloudfoundry/dependabot/go_modules/code.cloudfoundry.org/volume-mount-options-0.3.0
  > Merge pull request #15 from cloudfoundry/go-mod-tidy-20200310211248
  > Add github token to have ability to create a PR
  > Manually bump nfsv3driver onto goshims 0.1.0
  > Manually bump nfsv3driver to volume-mount-options v0.1.0
  > Manually bump nfsv3driver onto volumedriver v0.1.0
  > Merge pull request #12 from cloudfoundry/dependabot/go_modules/github.com/onsi/ginkgo-1.12.0
  > Merge pull request #11 from cloudfoundry/dependabot/go_modules/github.com/onsi/gomega-1.9.0
  > Hardcode uniqueVolumeIds to true
  > Add Makefile for running specs
  > Fix docker driver integration tests to no longer allow default-in-mount, allowed-in-mount options
  > Remove allowed-in-mount and default-in-mount configurations
  > remove useless/misleading "intr" option from list of nfs mount options
  > go module bump
  > Code refactor
  > Bump volumedriver
  > Run go mod vendor and go mod tidy
  > Bump volume driver and use new Invoke API
  > use the latest volume-mount-options
  > update vendored modules
  > Merge pull request #10 from cloudfoundry/dependabot/go_modules/github.com/onsi/gomega-1.8.1
  > Merge pull request #8 from cloudfoundry/dependabot/go_modules/github.com/onsi/gomega-1.8.0
  > Merge pull request #9 from cloudfoundry/dependabot/go_modules/github.com/onsi/ginkgo-1.11.0
  > Bump volumedriver
  > Refactor
  > Refactor
  > Bump volumedriver
  > correct script for running docker driver integration tests
  > Bump volumedriver
  > Refactor - Use new pgInvoker
  > remove unnecessary GO111MODULE setting [#169684821](https://www.pivotaltracker.com/story/show/169684821)
  > minor code clean up in integration test script [#169684821](https://www.pivotaltracker.com/story/show/169684821)
  > remove vestigial cruft from the README
  > build mapfs directly, instead of in mapfs-release when integration testing [#169684821](https://www.pivotaltracker.com/story/show/169684821)
  > add integration tests that run directly from this repo [#169684821](https://www.pivotaltracker.com/story/show/169684821)
  > add scripts to run units directly from this repo [#169684821](https://www.pivotaltracker.com/story/show/169684821)
  > Remove unused mount option 'fsname'
  > Bump volume mount options
  > Use vendored modules for suite test
  > Merge pull request #6 from cloudfoundry/dependabot/go_modules/github.com/onsi/ginkgo-1.10.3
  > Merge pull request #7 from cloudfoundry/dependabot/go_modules/github.com/onsi/gomega-1.7.1
julian-hj added a commit that referenced this issue Feb 19, 2021
@julian-hj
bump nfsv3driver
0d9a50c 
Submodule src/code.cloudfoundry.org/nfsv3driver 1fed479...ef3cf74:
  > Ignore minimum TLS warning
  < go mod vendor
  < Merge pull request #46 from cloudfoundry/dependabot/go_modules/code.cloudfoundry.org/volumedriver-0.14.0
  < go mod vendor
  < Merge pull request #45 from cloudfoundry/dependabot/go_modules/github.com/onsi/ginkgo-1.15.0
  < go mod vendor
  < Merge pull request #44 from cloudfoundry/dependabot/go_modules/github.com/onsi/gomega-1.10.5
  < go mod vendor
  < Merge pull request #43 from cloudfoundry/dependabot/go_modules/code.cloudfoundry.org/volumedriver-0.12.0
  < Revert "Add harbor cache for docker rate limiting"
  < Revert "Add harbor cache for docker rate limiting"
  < go mod vendor
  < Merge pull request #42 from cloudfoundry/dependabot/go_modules/code.cloudfoundry.org/volumedriver-0.11.0
  < Add harbor cache for docker rate limiting
  < Add harbor cache for docker rate limiting
  < go mod vendor
  < Merge pull request #41 from cloudfoundry/dependabot/go_modules/github.com/onsi/gomega-1.10.4
  < go mod vendor
  < Merge pull request #40 from cloudfoundry/dependabot/go_modules/github.com/onsi/gomega-1.10.3
  < go mod vendor
  < Merge pull request #39 from cloudfoundry/dependabot/go_modules/github.com/onsi/ginkgo-1.14.2
  < go mod vendor
  < Merge pull request #38 from cloudfoundry/dependabot/go_modules/github.com/onsi/gomega-1.10.2
  < go mod vendor
  < Merge pull request #37 from cloudfoundry/dependabot/go_modules/github.com/onsi/ginkgo-1.14.1
  < go mod vendor
  < Merge pull request #36 from cloudfoundry/dependabot/go_modules/github.com/onsi/ginkgo-1.14.0
  < go mod vendor
  < Merge pull request #35 from cloudfoundry/dependabot/go_modules/github.com/onsi/ginkgo-1.13.0
  < go mod vendor
  < Merge pull request #34 from cloudfoundry/dependabot/go_modules/github.com/onsi/ginkgo-1.12.3
  < go mod vendor
  < Merge pull request #33 from cloudfoundry/dependabot/go_modules/code.cloudfoundry.org/volumedriver-0.10.0
  < go mod vendor
  < Merge pull request #32 from cloudfoundry/dependabot/go_modules/github.com/onsi/ginkgo-1.12.2
  < go mod vendor
  < Merge pull request #30 from cloudfoundry/dependabot/go_modules/code.cloudfoundry.org/volumedriver-0.9.0
  < go mod vendor
  < Merge pull request #29 from cloudfoundry/dependabot/go_modules/github.com/onsi/ginkgo-1.12.1
  < go mod vendor
  < Merge pull request #25 from cloudfoundry/dependabot/go_modules/code.cloudfoundry.org/volumedriver-0.7.0
  < go mod vendor
  < Merge pull request #26 from cloudfoundry/dependabot/go_modules/code.cloudfoundry.org/goshims-0.4.0
  < go mod vendor
  < Merge pull request #24 from cloudfoundry/dependabot/go_modules/github.com/onsi/gomega-1.10.0
  > compatibility changes to work with newer volume-mount-options [#172568984](https://www.pivotaltracker.com/story/show/172568984)
  > Implemented a cache option
  < Implemented a cache option
  < go mod vendor
  < Merge pull request #22 from cloudfoundry/dependabot/go_modules/code.cloudfoundry.org/volumedriver-0.5.0
  < 🤖 go mod vendor
  < Merge pull request #21 from cloudfoundry/dependabot/go_modules/code.cloudfoundry.org/goshims-0.3.0
  < 🤖 go mod vendor
  < Merge pull request #20 from cloudfoundry/dependabot/go_modules/code.cloudfoundry.org/volumedriver-0.4.0
  < go mod vendor
  < Update go dependencies
  < Remove github actions
  < Merge pull request #19 from cloudfoundry/go-mod-tidy-20200311213050
  < Merge pull request #17 from cloudfoundry/dependabot/go_modules/code.cloudfoundry.org/volumedriver-0.3.0
  < Merge pull request #18 from cloudfoundry/go-mod-tidy-20200311212549
  < Merge pull request #16 from cloudfoundry/dependabot/go_modules/code.cloudfoundry.org/volume-mount-options-0.3.0
  < Merge pull request #15 from cloudfoundry/go-mod-tidy-20200310211248
  < Add github token to have ability to create a PR
  < Manually bump nfsv3driver onto goshims 0.1.0
  < Manually bump nfsv3driver to volume-mount-options v0.1.0
  < Manually bump nfsv3driver onto volumedriver v0.1.0
  < Merge pull request #12 from cloudfoundry/dependabot/go_modules/github.com/onsi/ginkgo-1.12.0
  < Merge pull request #11 from cloudfoundry/dependabot/go_modules/github.com/onsi/gomega-1.9.0
  < Hardcode uniqueVolumeIds to true
  < Add Makefile for running specs
  < Fix docker driver integration tests to no longer allow default-in-mount, allowed-in-mount options
  < Remove allowed-in-mount and default-in-mount configurations
  < remove useless/misleading "intr" option from list of nfs mount options
  < go module bump
  < Code refactor
  < Bump volumedriver
  < Run go mod vendor and go mod tidy
  < Bump volume driver and use new Invoke API
  < use the latest volume-mount-options
  < update vendored modules
  < Merge pull request #10 from cloudfoundry/dependabot/go_modules/github.com/onsi/gomega-1.8.1
  < Merge pull request #8 from cloudfoundry/dependabot/go_modules/github.com/onsi/gomega-1.8.0
  < Merge pull request #9 from cloudfoundry/dependabot/go_modules/github.com/onsi/ginkgo-1.11.0
  < Bump volumedriver
  < Refactor
  < Refactor
  < Bump volumedriver
  < correct script for running docker driver integration tests
  < Bump volumedriver
  < Refactor - Use new pgInvoker
  < remove unnecessary GO111MODULE setting [#169684821](https://www.pivotaltracker.com/story/show/169684821)
  < minor code clean up in integration test script [#169684821](https://www.pivotaltracker.com/story/show/169684821)
  < remove vestigial cruft from the README
  < build mapfs directly, instead of in mapfs-release when integration testing [#169684821](https://www.pivotaltracker.com/story/show/169684821)
  < add integration tests that run directly from this repo [#169684821](https://www.pivotaltracker.com/story/show/169684821)
  < add scripts to run units directly from this repo [#169684821](https://www.pivotaltracker.com/story/show/169684821)
  < Remove unused mount option 'fsname'
  < Bump volume mount options
  < Use vendored modules for suite test
  < Merge pull request #6 from cloudfoundry/dependabot/go_modules/github.com/onsi/ginkgo-1.10.3
  < Merge pull request #7 from cloudfoundry/dependabot/go_modules/github.com/onsi/gomega-1.7.1
  < Migrate to use go modules
julian-hj pushed a commit that referenced this issue Feb 19, 2021
bump nfsv3driver
edcb399 
Submodule src/code.cloudfoundry.org/nfsv3driver ef3cf74...1bf3c87:
  < Ignore minimum TLS warning
  > Ignore minimum TLS warning
  > go mod vendor
  > Merge pull request #46 from cloudfoundry/dependabot/go_modules/code.cloudfoundry.org/volumedriver-0.14.0
  > go mod vendor
  > Merge pull request #45 from cloudfoundry/dependabot/go_modules/github.com/onsi/ginkgo-1.15.0
  > go mod vendor
  > Merge pull request #44 from cloudfoundry/dependabot/go_modules/github.com/onsi/gomega-1.10.5
  > go mod vendor
  > Merge pull request #43 from cloudfoundry/dependabot/go_modules/code.cloudfoundry.org/volumedriver-0.12.0
  > Revert "Add harbor cache for docker rate limiting"
  > Revert "Add harbor cache for docker rate limiting"
  > go mod vendor
  > Merge pull request #42 from cloudfoundry/dependabot/go_modules/code.cloudfoundry.org/volumedriver-0.11.0
  > Add harbor cache for docker rate limiting
  > Add harbor cache for docker rate limiting
  > go mod vendor
  > Merge pull request #41 from cloudfoundry/dependabot/go_modules/github.com/onsi/gomega-1.10.4
  > go mod vendor
  > Merge pull request #40 from cloudfoundry/dependabot/go_modules/github.com/onsi/gomega-1.10.3
  > go mod vendor
  > Merge pull request #39 from cloudfoundry/dependabot/go_modules/github.com/onsi/ginkgo-1.14.2
  > go mod vendor
  > Merge pull request #38 from cloudfoundry/dependabot/go_modules/github.com/onsi/gomega-1.10.2
  > go mod vendor
  > Merge pull request #37 from cloudfoundry/dependabot/go_modules/github.com/onsi/ginkgo-1.14.1
  > go mod vendor
  > Merge pull request #36 from cloudfoundry/dependabot/go_modules/github.com/onsi/ginkgo-1.14.0
  > go mod vendor
  > Merge pull request #35 from cloudfoundry/dependabot/go_modules/github.com/onsi/ginkgo-1.13.0
  > go mod vendor
  > Merge pull request #34 from cloudfoundry/dependabot/go_modules/github.com/onsi/ginkgo-1.12.3
  > go mod vendor
  > Merge pull request #33 from cloudfoundry/dependabot/go_modules/code.cloudfoundry.org/volumedriver-0.10.0
  > go mod vendor
  > Merge pull request #32 from cloudfoundry/dependabot/go_modules/github.com/onsi/ginkgo-1.12.2
  > go mod vendor
  > Merge pull request #30 from cloudfoundry/dependabot/go_modules/code.cloudfoundry.org/volumedriver-0.9.0
  > go mod vendor
  > Merge pull request #29 from cloudfoundry/dependabot/go_modules/github.com/onsi/ginkgo-1.12.1
  > go mod vendor
  > Merge pull request #25 from cloudfoundry/dependabot/go_modules/code.cloudfoundry.org/volumedriver-0.7.0
  > go mod vendor
  > Merge pull request #26 from cloudfoundry/dependabot/go_modules/code.cloudfoundry.org/goshims-0.4.0
  > go mod vendor
  > Merge pull request #24 from cloudfoundry/dependabot/go_modules/github.com/onsi/gomega-1.10.0
  < compatibility changes to work with newer volume-mount-options [#172568984](https://www.pivotaltracker.com/story/show/172568984)
  < Implemented a cache option
  > Implemented a cache option
  > go mod vendor
  > Merge pull request #22 from cloudfoundry/dependabot/go_modules/code.cloudfoundry.org/volumedriver-0.5.0
  > 🤖 go mod vendor
  > Merge pull request #21 from cloudfoundry/dependabot/go_modules/code.cloudfoundry.org/goshims-0.3.0
  > 🤖 go mod vendor
  > Merge pull request #20 from cloudfoundry/dependabot/go_modules/code.cloudfoundry.org/volumedriver-0.4.0
  > go mod vendor
  > Update go dependencies
  > Remove github actions
  > Merge pull request #19 from cloudfoundry/go-mod-tidy-20200311213050
  > Merge pull request #17 from cloudfoundry/dependabot/go_modules/code.cloudfoundry.org/volumedriver-0.3.0
  > Merge pull request #18 from cloudfoundry/go-mod-tidy-20200311212549
  > Merge pull request #16 from cloudfoundry/dependabot/go_modules/code.cloudfoundry.org/volume-mount-options-0.3.0
  > Merge pull request #15 from cloudfoundry/go-mod-tidy-20200310211248
  > Add github token to have ability to create a PR
  > Manually bump nfsv3driver onto goshims 0.1.0
  > Manually bump nfsv3driver to volume-mount-options v0.1.0
  > Manually bump nfsv3driver onto volumedriver v0.1.0
  > Merge pull request #12 from cloudfoundry/dependabot/go_modules/github.com/onsi/ginkgo-1.12.0
  > Merge pull request #11 from cloudfoundry/dependabot/go_modules/github.com/onsi/gomega-1.9.0
  > Hardcode uniqueVolumeIds to true
  > Add Makefile for running specs
  > Fix docker driver integration tests to no longer allow default-in-mount, allowed-in-mount options
  > Remove allowed-in-mount and default-in-mount configurations
  > remove useless/misleading "intr" option from list of nfs mount options
  > go module bump
  > Code refactor
  > Bump volumedriver
  > Run go mod vendor and go mod tidy
  > Bump volume driver and use new Invoke API
  > use the latest volume-mount-options
  > update vendored modules
  > Merge pull request #10 from cloudfoundry/dependabot/go_modules/github.com/onsi/gomega-1.8.1
  > Merge pull request #8 from cloudfoundry/dependabot/go_modules/github.com/onsi/gomega-1.8.0
  > Merge pull request #9 from cloudfoundry/dependabot/go_modules/github.com/onsi/ginkgo-1.11.0
  > Bump volumedriver
  > Refactor
  > Refactor
  > Bump volumedriver
  > correct script for running docker driver integration tests
  > Bump volumedriver
  > Refactor - Use new pgInvoker
  > remove unnecessary GO111MODULE setting [#169684821](https://www.pivotaltracker.com/story/show/169684821)
  > minor code clean up in integration test script [#169684821](https://www.pivotaltracker.com/story/show/169684821)
  > remove vestigial cruft from the README
  > build mapfs directly, instead of in mapfs-release when integration testing [#169684821](https://www.pivotaltracker.com/story/show/169684821)
  > add integration tests that run directly from this repo [#169684821](https://www.pivotaltracker.com/story/show/169684821)
  > add scripts to run units directly from this repo [#169684821](https://www.pivotaltracker.com/story/show/169684821)
  > Remove unused mount option 'fsname'
  > Bump volume mount options
  > Use vendored modules for suite test
  > Merge pull request #6 from cloudfoundry/dependabot/go_modules/github.com/onsi/ginkgo-1.10.3
  > Merge pull request #7 from cloudfoundry/dependabot/go_modules/github.com/onsi/gomega-1.7.1
  > Migrate to use go modules
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
accepted
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@youngm @cf-gitbot @julian-hj