Python Devcontainer fails to start about airbyte HOT 2 CLOSED

@imrehg - Thanks for posting this. We are experimenting with these devcontainers as a means of streamlining community contributions as well as maintainer's code review processes. Your proposals to bump the version sound good to me. If you are able to contribute a PR, please tag me as reviewer and I'll happy review and approve.

We could also consider reverting to the "universal" image that Github/Microsoft are maintaining - while it is a larger image, it comes pre-loaded with Python and Node (to host VS Code and its extensions) and the universal image appears to be cached on runners and has a faster build times. (Not based on controlled experiments, just based on anecdotal observations.) On the negative side, I've had sporadic problems git committing (PGP error) on the universal image, although that may have been a one-time fluke.

Long-story short, we welcome your suggestions and I appreciate your posting this issue. Lmk how I can help! 🙏

from airbyte.

@aaronsteers I've submitted a PR with the proposed second option, to minimise the changes and focus on making the environment better fit for purpose.

Now that I can run the dev container, I can see that that the actual developer flow is indeed quite experimental in there, partly because it's new to me (so I'm not sure if everything has been set up correctly), and partly because it seems like the container does make the various config more mixed up (mapping config from the dev machine into the cotnainer).

As some illustrative examples, I use ssh key signing, and most config is loaded fine except the allowed signing keys file, since it's referring to the location outside of the container. Also had some issues getting the pre-commit hook work (I didn't). These bits are something to explore in a separate PR, probably. And that motivated me to keep this, rather than switching to the universal image, as I wouldn't be sure which issues that I encounter is due to the image change versus all the other setup. I can see that move would simplify the setup though, so likely it's a fitting suggestion for later.

from airbyte.