Skip to content

The future of community modules (the testcontainers-* packages) #412

Closed

Description

@alexanderankin

The testcontainers-* packages, e.g.:

were added as part of the last release, but it introduces a lot of complexity, for what could be just a lot of optional dependency groups in one package.

As it stands,
the dependency tree is something like:

testcontainers
+--- testcontainers-core -> *
+--- [postgres] (to represent "group" if testcontainers[postgres] or testcontainers[minio,postgres] is requested to be installed)
     +--- testcontainers-postgres (*)
          +--- testcontainers-core (*)
          +--- sqlalchemy (*)
          \--- psycopg2-binary (*)

but it could look like:

testcontainers
+--- [postgres] (to represent "group" if testcontainers[postgres] or testcontainers[minio,postgres] is requested to be installed)
     +--- sqlalchemy (*)
     \--- psycopg2-binary (*)

downsides:

  • of course users may get import errors, and we should then wrap those to educate users on how to install testcontainers[postgres] rather than just testcontainers.
  • temporary pain while we lack 100% parity with testcontainers-java (log + port waiting, rather than importing sqlalchemy)

upsides:

  • better developer experience on both sides (library and client side)
  • we can publish a single package to pypi, we still have yet to hear back from Till, and he is the only maintainer on those packages.

Activity

  1. alexanderankin commented on Jan 15, 2024

    @alexanderankin
    MemberAuthor

    all three currently available maintainers have a consensus (2 strongly, and one slightly) leaning towards the consolidated package approach but I wanted to make this decision publicly recorded in github issues, as well as to let people weigh in for consideration as well. thank you.

  2. pinned this issue on Jan 26, 2024
  3. changed the title [-]Decide what to do with the `testcontainers-*` packages[/-] [+]The future of community modules (the `testcontainers-*` packages)[/+] on Jan 26, 2024
  4. totallyzen commented on Jan 26, 2024

    @totallyzen
    Contributor

    hey @alexanderankin I've modified the title and pinned this as a major issue.
    We need to talk more about this with everyone as it isn't just about package layout and dependencies, it also means we have a LOT of exponential testing to do and that is becoming prohibitive with the costs to running those.

    Comments on what to do are welcome!

  5. covatic-john commented on Feb 14, 2024

    @covatic-john

    Hey happy to help where I can

  6. totallyzen commented on Feb 16, 2024

    @totallyzen
    Contributor

    Hey @covatic-john thanks for the offer! We'll definitnely take that up! I need to wrap my head around how we can split some of the work.

    • We're fairly settled to have only one package with extras at this point. It's just too complex to manage PyPI otherwise.
    • What remains is increasing quality and decide on what "official" container flavours to support and how.
    • Specifically the work with mypy is a pain in the back because of the many modules we already have.
    • Splitting the work by modules might help to parallelise the work.

    In addition, having chatted with Testcontainers staff they are also keen on reducing the number of "official" modules and try to make the generic containers flexible and easy to use (improving wait conditions for example).

    If you fancy comparing tc-java, tc-go and tc-python for features it would be awesome as it gives us some ideas on what to aim for. To be clear, no expectations, just giving you ideas on what you could help with.

    Cheers!

  7. covatic-john commented on Feb 16, 2024

    @covatic-john
  8. locked and limited conversation to collaborators on Mar 24, 2024
  9. converted this issue into a discussion #502 on Mar 24, 2024
  10. unpinned this issue on Aug 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions