FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Adding bigtable-emulator tox environment. by dhermes · Pull Request #2322 · googleapis/google-cloud-python · GitHub

/ google-cloud-python Public

Adding bigtable-emulator tox environment. - #2322

Merged
dhermes merged 1 commit into
googleapis:masterfrom
dhermes:add-bigtable-emulator
Sep 20, 2016
ConversationCommits (1)ChecksFiles changed
Merged

Adding bigtable-emulator tox environment.#2322
dhermes merged 1 commit into
googleapis:masterfrom
dhermes:add-bigtable-emulator

Conversation

dhermes commented Sep 15, 2016

Copy link
Copy Markdown
Contributor

Also updating the pubsub system test with similar support for detection of the emulator at run-time rather than import time.

Fixes #2242.

FYI @garye this is the current output:

$ tox -e bigtable-emulator
GLOB sdist-make: .../google-cloud-python/setup.py
bigtable-emulator inst-nodeps: .../google-cloud-python/.tox/dist/google-cloud-0.18.0.zip
bigtable-emulator installed: enum34==1.1.6,future==0.15.2,futures==3.0.5,gax-google-logging-v2==0.8.1,gax-google-pubsub-v1==0.8.1,google-cloud==0.18.0,google-gax==0.12.5,googleapis-common-protos==1.3.4,grpc-google-logging-v2==0.8.1,grpc-google-pubsub-v1==0.8.1,grpcio==1.0.0,httplib2==0.9.2,oauth2client==3.0.0,ply==3.8,protobuf==3.0.0,psutil==4.3.1,py==1.4.31,pyasn1==0.1.9,pyasn1-modules==0.0.8,pytest==3.0.2,rsa==3.4.2,six==1.10.0
bigtable-emulator runtests: PYTHONHASHSEED='2409315821'
bigtable-emulator runtests: commands[0] | python .../google-cloud-python/system_tests/run_emulator.py --package=bigtable
test_read_row (bigtable.TestDataAPI) ... ok
test_read_rows (bigtable.TestDataAPI) ... ok
test_read_with_label_applied (bigtable.TestDataAPI) ... skipped 'Labels not supported by Bigtable emulator'
test_create_instance (bigtable.TestInstanceAdminAPI) ... skipped 'Instance Admin API not supported in Bigtable emulator'
test_list_instances (bigtable.TestInstanceAdminAPI) ... skipped 'Instance Admin API not supported in Bigtable emulator'
test_reload (bigtable.TestInstanceAdminAPI) ... skipped 'Instance Admin API not supported in Bigtable emulator'
test_update (bigtable.TestInstanceAdminAPI) ... skipped 'Instance Admin API not supported in Bigtable emulator'
test_create_column_family (bigtable.TestTableAdminAPI) ... ok
test_create_table (bigtable.TestTableAdminAPI) ... ok
test_delete_column_family (bigtable.TestTableAdminAPI) ... ok
test_list_tables (bigtable.TestTableAdminAPI) ... ok
test_update_column_family (bigtable.TestTableAdminAPI) ... ok

----------------------------------------------------------------------
Ran 12 tests in 0.018s

OK (skipped=5)
E0915 09:38:34.312954162    9877 network_status_tracker.c:48] Memory leaked as all network endpoints were not shut down
___________________________________________________________________ summary ___________________________________________________________________
  bigtable-emulator: commands succeeded
  congratulations :)

dhermes added testing api: bigtable Issues related to the Bigtable API. labels Sep 15, 2016
googlebot added the cla: yes This human has signed the Contributor License Agreement. label Sep 15, 2016
dhermes mentioned this pull request Sep 15, 2016
Closed

garye commented Sep 15, 2016

Copy link
Copy Markdown
Contributor

LGTM, this is the right set of stuff to be skipping.

dhermes commented Sep 15, 2016

Copy link
Copy Markdown
Contributor Author

w00t.

daspecster reviewed Sep 15, 2016
View reviewed changes
Comment thread tox.ini Outdated
python {toxinidir}/system_tests/run_emulator.py --package=bigtable
passenv =
GOOGLE_CLOUD_DISABLE_GRPC
deps = {[testenv:datastore-emulator]deps}

This comment was marked as spam.

Sign in to view

This comment was marked as spam.

Sign in to view

This comment was marked as spam.

Sign in to view

Comment thread system_tests/run_emulator.py Outdated
wait_ready_prefix(popen, _PS_READY_LINE_PREFIX)


def bigtable_wait_ready(popen):

This comment was marked as spam.

Sign in to view

This comment was marked as spam.

Sign in to view

This comment was marked as spam.

Sign in to view

Comment thread system_tests/run_emulator.py
}
_DS_READY_LINE = '[datastore] Dev App Server is now running.\n'
_PS_READY_LINE_PREFIX = '[pubsub] INFO: Server started, listening on '
_BT_READY_LINE_PREFIX = '[bigtable] Cloud Bigtable emulator running on '

This comment was marked as spam.

Sign in to view

This comment was marked as spam.

Sign in to view

tseaver reviewed Sep 15, 2016
View reviewed changes
Comment thread system_tests/bigtable.py
if Config.IN_EMULATOR is not None:
self.skipTest(
'Instance Admin API not supported in Bigtable emulator')
self.instances_to_delete = []

This comment was marked as spam.

Sign in to view

This comment was marked as spam.

Sign in to view

This comment was marked as spam.

Sign in to view

dhermes force-pushed the add-bigtable-emulator branch from afc36e2 to c877adf Compare September 16, 2016 17:15

dhermes commented Sep 16, 2016

Copy link
Copy Markdown
Contributor Author

@daspecster @tseaver PTAL. (Also on LGTM, please remind me to squash.)

daspecster commented Sep 16, 2016

Copy link
Copy Markdown
Contributor

LGTM unless @tseaver has more.

dhermes force-pushed the add-bigtable-emulator branch from c877adf to 6805312 Compare September 20, 2016 19:41
Adding bigtable-emulator tox environment.
b42bdc8 
Also updating the pubsub system test with similar support
for detection of the emulator at run-time rather than
import time.

Fixes googleapis#2242.
dhermes force-pushed the add-bigtable-emulator branch from 6805312 to b42bdc8 Compare September 20, 2016 19:41
dhermes merged commit f5fb643 into googleapis:master Sep 20, 2016
dhermes deleted the add-bigtable-emulator branch September 20, 2016 20:02

garye commented Sep 20, 2016

Copy link
Copy Markdown
Contributor

Glad to see this merged! Can we do a release on the back of this? We want to add emulator documentation that mentions the python support.

dhermes commented Sep 20, 2016

Copy link
Copy Markdown
Contributor Author

@garye #2242 is the real support needed, and that made it in 0.19.0 (released yesterday).

This change just makes sure maintainers (or any people who clone the repo) can periodicially check if the emulator is working.

garye commented Sep 20, 2016

Copy link
Copy Markdown
Contributor

Good to know, thanks @dhermes.

dhermes commented Sep 20, 2016

Copy link
Copy Markdown
Contributor Author

Sure thing

parthea pushed a commit that referenced this pull request Mar 2, 2026
chore: librarian release pull request: 20251212T151524Z (#2344)
9d14bda 
PR created by the Librarian CLI to initialize a release. Merging this PR
will auto trigger a release.

Librarian Version: v0.7.0
Language Image:
us-central1-docker.pkg.dev/cloud-sdk-librarian-prod/images-prod/python-librarian-generator@sha256:c8612d3fffb3f6a32353b2d1abd16b61e87811866f7ec9d65b59b02eb452a620
<details><summary>google-cloud-bigquery: 3.39.0</summary>

##
[3.39.0](googleapis/python-bigquery@v3.38.0...v3.39.0)
(2025-12-12)

### Features

* adds support for Python runtime 3.14 (#2322)
([6065e14c](googleapis/python-bigquery@6065e14c))

* Add ExternalRuntimeOptions to BigQuery routine (#2311)
([fa76e310](googleapis/python-bigquery@fa76e310))

### Bug Fixes

* remove ambiguous error codes from query retries (#2308)
([8bbd3d01](googleapis/python-bigquery@8bbd3d01))

* include `io.Base` in the `PathType` (#2323)
([b11e09cb](googleapis/python-bigquery@b11e09cb))

* honor custom `retry` in `job.result()` (#2302)
([e118b029](googleapis/python-bigquery@e118b029))

### Documentation

* remove experimental annotations from GA features (#2303)
([1f1f9d41](googleapis/python-bigquery@1f1f9d41))

</details>

Co-authored-by: Daniel Sanche <d.sanche14@gmail.com>
parthea pushed a commit that referenced this pull request Mar 6, 2026
feat: adds support for Python runtime 3.14 (#2322)
c5b8872 
* feat: adds support for Python runtime 3.14

* adds step to install gdal

* adds files required by pyarrow

* adds repo required by pyarrow

* corrects url to repo required by pyarrow

* testing a theory with a conditional

* testing a theory with a conditional version of ubuntu

* testing a new approach to installing arrow

* testing a new approach to dearmoring the key

* back to the basics

* trying a conditional again.

* adds explanatory comment resets ubuntu version to latest

* Apply suggestion from @chalmerlowe

* Apply suggestion from @chalmerlowe

* Apply suggestion from @chalmerlowe

* Apply suggestion from @chalmerlowe
parthea pushed a commit that referenced this pull request Apr 1, 2026
chore: Add conventional-pre-commit hook (#2322)
1354937 
Added conventional-pre-commit hook for commit-msg stage.

Thank you for opening a Pull Request! Before submitting your PR, there
are a few things you can do to make sure it goes smoothly:
- [ ] Make sure to open an issue as a
[bug/issue](https://github.com/googleapis/python-bigquery-dataframes/issues/new/choose)
before writing your code! That way we can discuss the change, evaluate
designs, and agree on the general idea
- [ ] Ensure the tests and linter pass
- [ ] Code coverage does not decrease (if any source code was changed)
- [ ] Appropriate docs were updated (if necessary)

Fixes #<issue_number_goes_here> 🦕

---------

Co-authored-by: Shenyang Cai <sycai@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Show hidden characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

2 more reviewers
daspecster daspecster left review comments
tseaver tseaver left review comments
Reviewers whose approvals may not affect merge requirements

Assignees

tseaver

Labels

api: bigtable Issues related to the Bigtable API. cla: yes This human has signed the Contributor License Agreement. testing

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

Back | FazBrowse Home | New Git URL