You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
Add support for BIGTABLE_EMULATOR_HOST environment variable #2242
👍 I just noticed last week that the Bigtable emulator had been added. I've been meaning to sort out some b0rken-ness with the other clients (never checked if they worked after adding gRPC support to the clients).
@garye I mostly have this finished but it seems like most of system_tests/bigtable.py fails because the backend comes back with UNAVAILABLE. Is there a list of methods actually working in the emulator?
Instance-related commands aren't implemented and I suspect that's what you're running into. It's meant for table-level admin/read/write operations, so you can reference any project/instance when you create a table.
OK. It seems empirically that deleting a table doesn't work. Also, I'm starting to think some of the UNAVAILABLE I'm seeing are masked as other issues, e.g. creating a table that already exists (since a previous delete failed). And possibly my system test has too high a QPS for the server?
UNAVAILABLE was because the server crashed when asked to do a ReadRows with no Rows specified. This should be supported so we'll fix ASAP, but it will be at least a week until a fix can make it into a gcloud release.
Deleting a table should work fine, it does locally when I ran a patched emulator that handles the ReadRows request.
The TableAdmin tests are supported and pass locally if enabled.
Reading with a label is not supported and isn't high on the list since it's seldom used. I'd skip that test in emulator mode.
Thanks for digging into this! We already skip the IAM tests on the Pub / Sub side when running in the emulator, so not a big deal for labels.
The sugar to allow local connections when the emulator already works before waiting for the gcloud CLI fix to be released. Worth adding that support and then adding a working tox -e bigtable-emulator in a subsequent PR?
Activity
dhermes commentedon Sep 2, 2016
👍 I just noticed last week that the Bigtable emulator had been added. I've been meaning to sort out some b0rken-ness with the other clients (never checked if they worked after adding gRPC support to the clients).
dhermes commentedon Sep 6, 2016
@garye I mostly have this finished but it seems like most of
system_tests/bigtable.pyfails because the backend comes back withUNAVAILABLE. Is there a list of methods actually working in the emulator?garye commentedon Sep 6, 2016
Instance-related commands aren't implemented and I suspect that's what you're running into. It's meant for table-level admin/read/write operations, so you can reference any project/instance when you create a table.
dhermes commentedon Sep 6, 2016
OK. It seems empirically that deleting a table doesn't work. Also, I'm starting to think some of the
UNAVAILABLEI'm seeing are masked as other issues, e.g. creating a table that already exists (since a previous delete failed). And possibly my system test has too high a QPS for the server?garye commentedon Sep 6, 2016
dhermes commentedon Sep 6, 2016
Thanks for digging into this! We already skip the IAM tests on the Pub / Sub side when running in the emulator, so not a big deal for labels.
The sugar to allow local connections when the emulator already works before waiting for the
gcloudCLI fix to be released. Worth adding that support and then adding a workingtox -e bigtable-emulatorin a subsequent PR?garye commentedon Sep 6, 2016
Yeah that works for me, I'd like to see something out sooner rather than later.
dhermes commentedon Sep 8, 2016
@garye Now that #2249 has been merged, can you give me a heads up when the fixes are in so I can add
tox -e bigtable-emulator?garye commentedon Sep 15, 2016
@dhermes The latest gcloud/emulator release should have the fix. Can you give it a shot?
Adding bigtable-emulator tox environment.
dhermes commentedon Sep 15, 2016
@garye Thanks for the heads-up. Check out #2322 and LMK if I've disable too many test cases.
Adding bigtable-emulator tox environment.
Adding bigtable-emulator tox environment.
Adding bigtable-emulator tox environment.
chore(deps): update all dependencies to v3.35.0 (#2242)
chore(main): release 2.29.0 (#2242)