Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bigtable: 'TestTableAdminAPI.test_exists' flakes on ghost table #6245

Closed
tseaver opened this issue Oct 17, 2018 · 1 comment
Closed

Bigtable: 'TestTableAdminAPI.test_exists' flakes on ghost table #6245

tseaver opened this issue Oct 17, 2018 · 1 comment
Assignees
Labels
api: bigtable Issues related to the Bigtable API. flaky testing type: process A process-related concern. May include testing, release, or the like.

Comments

@tseaver
Copy link
Contributor

tseaver commented Oct 17, 2018

From Kokoro today:

________________________ TestTableAdminAPI.test_exists _________________________

self = <tests.system.TestTableAdminAPI testMethod=test_exists>

    def test_exists(self):
        temp_table_id = 'test-table_existence'
        temp_table = Config.INSTANCE.table(temp_table_id)
        self.assertFalse(temp_table.exists())
        temp_table.create()
        self.assertTrue(temp_table.exists())
        temp_table.delete()
>       self.assertFalse(temp_table.exists())
E       AssertionError: True is not false

Presumably, this is an eventual consistency thing.

@tseaver tseaver added testing api: bigtable Issues related to the Bigtable API. type: process A process-related concern. May include testing, release, or the like. flaky labels Oct 17, 2018
@tseaver
Copy link
Contributor Author

tseaver commented Oct 24, 2018

Another just now.

tseaver added a commit that referenced this issue Nov 1, 2018
* Harden table existence checks for eventual consistency.

Closes #6245.

* Snippet: require fewer serve nodes when updating cluster.

Avoid errors due to quota with simultaneous CI runs.

Closes #6366.

* Avoid use of 'is' keyword in asserts where identity is not promised.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api: bigtable Issues related to the Bigtable API. flaky testing type: process A process-related concern. May include testing, release, or the like.
Projects
None yet
Development

No branches or pull requests

1 participant