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

WIP: Reproduce bigtable issue #1780

Open
wants to merge 34 commits into
base: master
Choose a base branch
from

Conversation

justinsb
Copy link
Collaborator

  • tfprovider: support intercepting GRPC requests
  • tests: record grpc requests in our golden tests
  • tests: capture golden output for bigtabletable test
  • mockgcp: Support for bigtable
  • mockgcp: Support BigtableTable
  • tests: capture golden output for bigtable
  • Create bigtable test
  • WIP: tests - try client-side update
  • tests: export BigtableInstance
  • powertool: change-state-into-spec can remove fields set by state-into-spec
  • tests: start to refactor normalization logic out of specific tests
  • tests: Add ability to run CLI to script_test
  • tests: Create change-state-into-spec powertool test scenario
  • tests: Create test scenario for force-set-field powertool
  • test with powertool
  • WIP
  • fix merge
  • tests
  • test with mocks
  • bigtable: better mockgcp
  • WIP: script test with client-side apply
  • capture generated real output
  • mockgcp: Generate grpc bindings for bigtable
  • mockgcp: better support for updateMask
  • mockgcp: more bigtable
  • mockgcp: bigtable can serve on http also
  • autogen: generated bigtable grpc-gateway bindings
  • direct actuation: BigtableInstance
  • undo managedFields change
  • Rebase fixes
  • Split out bigtable testfixutes
  • mockgcp: better handling of invalid bigtable cluster configuration
  • test passing by appending & state-into-spec:absent
  • failing tests

We aim to construct a readable version of the GRPC requests.
This uses our new GRPC collection infrastructure.
This is tricky because our current is actually not using REST
(uniquely so!), so our normal interception method doesn't work.
Note this service is implemented via GRPC, so we need some more complex interception.
…-spec

Using the field manager, we can detect and remove any fields that were
set by state-into-spec.
This way our various tests can share one normalization function
This enables us to test our powertools in our existing framework
Changing the location of a StorageBucket is a good example usage.
Trying something different: grpc binding to the "real" google protos.
Filter both body & query parameters, to map json style field-names to
proto fields.
This uses the grpc-gateway generated code only
A slightly complex resource because of the Cluster subresources
Copy link
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please ask for approval from justinsb. For more information see the Kubernetes Code Review Process.

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant