Integrate Oracle Test Pilot into CI

[Tim Schilling]

As per this ​forum post, there's a new tool from Oracle that's being built which is designed to help open source projects support Oracle.

The goal here is to be able to run the oracle tests with GitHub Actions using Oracle Test Pilot. Based on that, the support and reliability, we can later investigate a complete migration to the tool.

[Loïc LEFEVRE]

Which workflow would be the best to target for Oracle Test Pilot integration?

I've tested with schedule_tests.yml by adding a new Job but please confirm.

[Jacob Walls]

Thanks, let's evaluate this.

Replying to Loïc LEFEVRE:

Which workflow would be the best to target for Oracle Test Pilot integration?

I've tested with schedule_tests.yml by adding a new Job but please confirm.

Yep, a new job next to the postgresql job seems logical. We can always port to the PR workflow later.

[Loïc LEFEVRE]

Hi Tim,
you can find an example of a job run here: ​https://github.com/loiclefevre/django-otp/actions/runs/26885786028/job/79297503877

I've just commented the other jobs for the time being.

Feel free to comment.
Thanks

[Tim Schilling]

The code looks pretty manageable! ​https://github.com/django/django/compare/main...loiclefevre:django-otp:otp

[Loïc LEFEVRE]

Quick question: is there any possibility that Connection(s) open to the test database(s) (defaut, other) may not be closed properly while running the tests? (at the very end, in the case there are no errors)

[Loïc LEFEVRE]

Reason for asking is that I've enabled Autonomous databases 19c and 26ai and the database cleanup process fail with following errors:

• ORA-01940: cannot drop a user that is currently connected
• ORA-01940: cannot drop a user that is currently connected

see: ​https://github.com/loiclefevre/django-otp/actions/runs/26998861078

[Loïc LEFEVRE]

Some updates:

• the issues with connections still in use after tests are finished is managed at the GitHub Action level now (the setup test pilot one)
• the PR phase 1 content can be seen here: ​https://github.com/django/django/compare/main...loiclefevre:django-otp:otp-integration-phase-1
  • remark: the commented jobs will not be commented in the final PR
  • I'm awaiting some review to be done to confirm it aligns with the requirements discussed last week
  • a job execution example can be seen here: ​https://github.com/loiclefevre/django-otp/actions/runs/28530651813/job/84578949702

[Loïc LEFEVRE]

And here is the PR: ​https://github.com/django/django/pull/21597

[Loïc LEFEVRE]

Comments provided within the PR review have been implemented. Only remains the one related to Spatial tests which I believe could be done through another ticket so that we don't mix topics :)

[Jacob Walls]

Replying to Loïc LEFEVRE:

Comments provided within the PR review have been implemented. Only remains the one related to Spatial tests which I believe could be done through another ticket so that we don't mix topics :)

Done, see #37136

[Jacob Walls]

Blocked pending a discussion with Ops Team about sandboxing.