-
Notifications
You must be signed in to change notification settings - Fork 6.6k
Adds updates including compute #2436
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
|
Now I remember - all of the CRUD methods started failing for this sample. As far as I can tell, the library updates should not affect the API calls. In fact, downgrading to the previous commit does not affect test results: Unable to determine how to set up the tests, I gave up on contributing fixes in this case. It's entirely possible there's a race condition or pre-sample setup that is missing. |
gguuss
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The CRUD tests in asset were almost definitely broken. I think the right thing to do is follow up internally with cwxie
|
Tracking down cwxie worked, the tests are now re-enabled and fixed! |
|
@tswast Any idea why the composer tests might be failing (e.g. API changes, not unexpected responses)? |
|
@gguuss It's due to a dependency problem in |
|
@tswast I tried pinning an older version of flask, still seeing: I'll see about downgrading to 1.10.2 |
|
@leahecole Gus's traceback seems to indicate there's a new Resources object in some Kubernetes logic that used to be a dictionary. Any ideas? Or should we just disable the kubernetespodoperator test for now? |
|
Ugh given that it's blocking other people, I'd say disable it and assign either an issue, a buganizer bug, or both to me to fix once I'm back from this trip. I'll be able to take a look at it the week of the 28th, but I don't want it to block folks before then. |
|
I found a "fix", which is to pass the dictionary from the pod resource object instead of the object itself. |
|
heh. Noted. @gguuss do you need any action from me at this point? If so, let me know and I'll prioritize it for Monday. |
|
I'll assign an issue for you in GH to follow up on before EOQ, no rush :)
…On Fri, Oct 18, 2019 at 7:13 AM Leah E. Cole ***@***.***> wrote:
heh. Noted. @gguuss <https://github.com/gguuss> do you need any action
from me at this point? If so, let me know and I'll prioritize it for Monday.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#2436?email_source=notifications&email_token=AA2VD6QNGPCJH4P4BH2VI2TQPHAAFA5CNFSM4I5TWEL2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEBUTMZA#issuecomment-543766116>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AA2VD6UBNAHCSS7GPNGFCKTQPHAAFANCNFSM4I5TWELQ>
.
--
Gus Class
email : [email protected]
phone: 425.200.4487
web : https://gusclass.com
|
|
@crwilcox friendly ping :) |
No description provided.