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

Continuous test runs haven't succeeded since 15 September #444

Closed
crwilcox opened this issue Oct 11, 2021 · 3 comments · Fixed by #445
Closed

Continuous test runs haven't succeeded since 15 September #444

crwilcox opened this issue Oct 11, 2021 · 3 comments · Fixed by #445
Assignees
Labels
api: bigtable Issues related to the googleapis/python-bigtable API. priority: p1 Important issue which blocks shipping the next release. Will be fixed prior to next release. type: bug Error or flaw in code with unintended results or allowing sub-optimal usage patterns.

Comments

@crwilcox
Copy link
Contributor

b75b9d1...83958df

nox > * system_emulated: failed

____________________ ERROR at setup of test_table_read_row _____________________

admin_client = <google.cloud.bigtable.client.Client object at 0x7f95fe9e6d00>
data_instance_id = 'g-c-p-d-1631732911078'
instance_labels = {'python-system': '2021-09-15t19-08-31'}
data_cluster_id = 'g-c-p-d-1631732911078-cluster', location_id = 'us-central1-c'
serve_nodes = 3, in_emulator = True

    @pytest.fixture(scope="session")
    def data_instance_populated(
        admin_client,
        data_instance_id,
        instance_labels,
        data_cluster_id,
        location_id,
        serve_nodes,
        in_emulator,
    ):
        # Emulator does not support instance admin operations (create / delete).
        # See: https://cloud.google.com/bigtable/docs/emulator
        if not in_emulator:
            instance = admin_client.instance(data_instance_id, labels=instance_labels)
            cluster = instance.cluster(
                data_cluster_id, location_id=location_id, serve_nodes=serve_nodes,
            )
            operation = instance.create(clusters=[cluster])
            operation.result(timeout=30)
    
>       yield instance
E       UnboundLocalError: local variable 'instance' referenced before assignment

tests/system/conftest.py:159: UnboundLocalError

@product-auto-label product-auto-label bot added the api: bigtable Issues related to the googleapis/python-bigtable API. label Oct 11, 2021
@crwilcox crwilcox self-assigned this Oct 11, 2021
@crwilcox crwilcox added the type: bug Error or flaw in code with unintended results or allowing sub-optimal usage patterns. label Oct 11, 2021
@crwilcox crwilcox added the priority: p1 Important issue which blocks shipping the next release. Will be fixed prior to next release. label Oct 11, 2021
@tseaver
Copy link
Contributor

tseaver commented Oct 12, 2021

@crwilcox Where are the test reports for those CI runs? If we could get #430 merged, we could see the emulator running in GH actions.

@tseaver
Copy link
Contributor

tseaver commented Oct 12, 2021

E.g., this GH action shows the emulator tests passing.

@crwilcox
Copy link
Contributor Author

crwilcox commented Oct 12, 2021

@crwilcox Where are the test reports for those CI runs? If we could get #430 merged, we could see the emulator running in GH actions.

Unfortunately I am unsure if there is a non-google view. We have a dashboard of history of Kokoro runs accessible.

Those are only passing due to the changes on that PR. previously, a unbound local error would occur due to the way instances were created by a fixture. Also, some tests that cannot run against the emulator needed to be skipped. (#445)

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 googleapis/python-bigtable API. priority: p1 Important issue which blocks shipping the next release. Will be fixed prior to next release. type: bug Error or flaw in code with unintended results or allowing sub-optimal usage patterns.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants