Summary

Fix introspection in rb-site upgrade.

Review Request #14901 — Created March 14, 2026 and submitted March 16, 2026, 3:29 p.m.

Information

Owner

david

Repository

Review Board

Branch

release-7.1.x

Bugs

Depends On

Reviewers

Groups

reviewboard

People

Description

At some point, django changed the name of get_key_columns in the

DatabaseIntrospection class to get_primary_key_columns. We never

noticed because we haven't had any failures during upgrades, but this

affects Django 4.2 as well.

Testing Done

Verified this change inside a CI upgrade tests container.

Commits

Summary	ID
Fix introspection in rb-site upgrade. At some point, django changed the name of `get_key_columns` in the `DatabaseIntrospection` class to `get_primary_key_columns`. We never because we haven't had any failures during upgrades, but this affects Django 4.2 as well. Testing Done: Verified this change inside a CI upgrade tests container.	ppxpkrntnqllnmtymzlzpolklulzyqwl

Issues

Description	From	Last Updated
Missing something in the description: "We never <?> because we ..."	chipx86	March 16, 2026, 3:14 p.m.

flake8 passed.

JSHint passed.

Ship it!

The issue has been resolved. Show all issues

Missing something in the description: "We never <?> because we ..."

Description:: At some point, django changed the name of get_key_columns in the
DatabaseIntrospection class to get_primary_key_columns. We never
~ because we haven't had any failures during upgrades, but this affects
~ Django 4.2 as well.
~ noticed because we haven't had any failures during upgrades, but this
~ affects Django 4.2 as well.

Status:: Completed

		At some point, django changed the name of `get_key_columns` in the
		`DatabaseIntrospection` class to `get_primary_key_columns`. We never
~		because we haven't had any failures during upgrades, but this affects
~		Django 4.2 as well.
	~	noticed because we haven't had any failures during upgrades, but this
	~	affects Django 4.2 as well.