Skip to content

Add rolling upgrade tests for table oids assigned on 6.3#410

Open
jeeminso wants to merge 2 commits intomasterfrom
jeeminso/check-table-oid
Open

Add rolling upgrade tests for table oids assigned on 6.3#410
jeeminso wants to merge 2 commits intomasterfrom
jeeminso/check-table-oid

Conversation

@jeeminso
Copy link
Copy Markdown
Contributor

Summary of the changes / Why this is an improvement

Checklist

  • Link to issue this PR refers to (if applicable): Fixes #???

Comment on lines +106 to +109
self.assertEqual(
zero_oids,
[],
f"pg_class contains oid = 0 for relations: {zero_oids}"
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

======================================================================
FAIL: test_rolling_upgrade_5_to_6 (test_rolling_upgrade.RollingUpgradeTest.test_rolling_upgrade_5_to_6) [6.2 -> 6.3.x]
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/var/lib/jenkins/workspace/CrateDB/qa/crate_qa_on_pr/tests/bwc/test_rolling_upgrade.py", line 40, in test_rolling_upgrade_5_to_6
    self._test_rolling_upgrade(path, nodes=3)
    ~~~~~~~~~~~~~~~~~~~~~~~~~~^^^^^^^^^^^^^^^
  File "/var/lib/jenkins/workspace/CrateDB/qa/crate_qa_on_pr/tests/bwc/test_rolling_upgrade.py", line 106, in _test_rolling_upgrade
    self.assertEqual(
    ~~~~~~~~~~~~~~~~^
        zero_oids,
        ^^^^^^^^^^
        [],
        ^^^
        f"pg_class contains oid = 0 for relations: {zero_oids}"
        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
    )
    ^
AssertionError: Lists differ: ['rx', 'x', 'y', 'parted', 't1', 't2', 't3'] != []

First list contains 7 additional elements.
First extra element 0:
'rx'

- ['rx', 'x', 'y', 'parted', 't1', 't2', 't3']
+ [] : pg_class contains oid = 0 for relations: ['rx', 'x', 'y', 'parted', 't1', 't2', 't3']

----------------------------------------------------------------------
Ran 1 test in 88.806s

currently all table oids are 0.

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.

1 participant