Skip to content

Avoid KeyError when attempting to access "properties" key#55

Open
larsks wants to merge 1 commit into
instrumenta:masterfrom
larsks:bug/54
Open

Avoid KeyError when attempting to access "properties" key#55
larsks wants to merge 1 commit into
instrumenta:masterfrom
larsks:bug/54

Conversation

@larsks

@larsks larsks commented Oct 22, 2021

Copy link
Copy Markdown

Closes #54

@spiarh

spiarh commented Nov 12, 2021

Copy link
Copy Markdown

hi @garethr , any chance to see this merged ? This fixed the issue we hit running this script against the openapi resources from OpenShift 4.8. This allowed the script to generate the shared definitions.

Thanks @larsks and @garethr

@ReDestroyDeR

Copy link
Copy Markdown

Hello @garethr, up on this PR. Issue #54 got a response from me and @paulcwarren and this commit is fixing it. When it's getting merged?

Thanks.

@bilbof

bilbof commented Apr 9, 2022

Copy link
Copy Markdown

I hit this issue with OpenShift 4.8 - can confirm the fix in the commit resolves it, thanks @larsks 👍

@larsks

larsks commented Mar 30, 2023

Copy link
Copy Markdown
Author

I'm collecting fixes in https://github.com/larsks/openapi2jsonschema. I've merged this patch into that repository.

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

Successfully merging this pull request may close these issues.

"KeyError: properties" when parsing OpenAPI spec from Kubernetes

4 participants