-
Notifications
You must be signed in to change notification settings - Fork 367
Add ability to set user on Tasks #4433
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
Conversation
8c9ee27 to
3fd097b
Compare
Changes in cloud_controller_ng:
- Add ability to set user on Tasks
PR: cloudfoundry/cloud_controller_ng#4433
Author: Tim Downey <tcdowney@users.noreply.github.com>
|
@tcdowney @Gerg We are seeing errors querying the task API for existing applications once this change is applied. Tasks still execute successfully but querying the task API fails. The error message is vague:
This only fails for apps that existed before the update and had tasks ran. I suspect this points to a nil/null issue related the missing user in the task records created before the update. Please let me know if there is additional info I can provide to help track this down. |
|
The underlying error should be available in the cloud controller logs. I wasn't able to reproduce this by deploying an older version, running a task, deploying a new version, and then listing tasks. I attempted to reproduce on an environment using MySQL; are you using Postgres? |
|
Thanks, I was able to find some logs. This impacted multiple customers before we rolled back. Yes, we are using postgres. |
|
We have been trying to recreate this issue in our dev environment and have not been able to yet. Oddly we saw this in our dev and prod environments at the same time. We are still working on it. |
|
I tried to reproduce with Postgres, and it still worked fine. My guess is it's something specific to those apps/tasks. There should be a stack trace in the logs coming from here. It should look something like: |
|
Having a heck of a time reproducing the problem, but DID see what Steve saw. Unfortunately the two apps that had the problem have either been deleted or redeployed which in each case made the problem go away. There are a few logs the next day with If we find anything more definitive we'll drop an update here. |
Issue: #4372
I have reviewed the contributing guide
I have viewed, signed, and submitted the Contributor License Agreement
I have made this pull request to the
mainbranchI have run all the unit tests using
bundle exec rake[ in progress ] I have run CF Acceptance Tests