-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
[vtctld] Migrate topo management RPCs #7395
Changes from 29 commits
c26306c
7175fbc
653d3f3
a7bb06e
e36b6b6
88a4431
9cfc46b
c5ed719
cd30750
8d7426f
4dd2a96
d58e478
9983e22
0e69a34
c379ca7
7e4a8be
5d8090c
d6cc15f
25e0a48
4e118c8
e713f57
c20a128
762a813
7030f62
74c4883
01560e4
1efbfca
b834775
24e4527
99dd25d
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,93 @@ | ||
/* | ||
Copyright 2021 The Vitess Authors. | ||
|
||
Licensed under the Apache License, Version 2.0 (the "License"); | ||
you may not use this file except in compliance with the License. | ||
You may obtain a copy of the License at | ||
|
||
http://www.apache.org/licenses/LICENSE-2.0 | ||
|
||
Unless required by applicable law or agreed to in writing, software | ||
distributed under the License is distributed on an "AS IS" BASIS, | ||
WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. | ||
See the License for the specific language governing permissions and | ||
limitations under the License. | ||
*/ | ||
|
||
package cli | ||
|
||
import ( | ||
"github.com/spf13/pflag" | ||
|
||
"vitess.io/vitess/go/flagutil" | ||
"vitess.io/vitess/go/vt/key" | ||
"vitess.io/vitess/go/vt/topo/topoproto" | ||
|
||
topodatapb "vitess.io/vitess/go/vt/proto/topodata" | ||
) | ||
|
||
// StringMapValue augements flagutil.StringMapValue so it can be used as a | ||
// pflag.Value. | ||
type StringMapValue struct { | ||
flagutil.StringMapValue | ||
} | ||
|
||
// Type is part of the pflag.Value interface. | ||
func (v *StringMapValue) Type() string { | ||
return "cli.StringMapValue" | ||
} | ||
|
||
// KeyspaceIDTypeFlag adds the pflag.Value interface to a | ||
// topodatapb.KeyspaceIdType. | ||
type KeyspaceIDTypeFlag topodatapb.KeyspaceIdType | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. where is this used? didn't see a reference in this PR There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. |
||
|
||
var _ pflag.Value = (*KeyspaceIDTypeFlag)(nil) | ||
|
||
// Set is part of the pflag.Value interface. | ||
func (v *KeyspaceIDTypeFlag) Set(arg string) error { | ||
t, err := key.ParseKeyspaceIDType(arg) | ||
if err != nil { | ||
return err | ||
} | ||
|
||
*v = KeyspaceIDTypeFlag(t) | ||
|
||
return nil | ||
} | ||
|
||
// String is part of the pflag.Value interface. | ||
func (v *KeyspaceIDTypeFlag) String() string { | ||
return key.KeyspaceIDTypeString(topodatapb.KeyspaceIdType(*v)) | ||
} | ||
|
||
// Type is part of the pflag.Value interface. | ||
func (v *KeyspaceIDTypeFlag) Type() string { | ||
return "cli.KeyspaceIdTypeFlag" | ||
} | ||
|
||
// KeyspaceTypeFlag adds the pflag.Value interface to a topodatapb.KeyspaceType. | ||
type KeyspaceTypeFlag topodatapb.KeyspaceType | ||
|
||
var _ pflag.Value = (*KeyspaceTypeFlag)(nil) | ||
|
||
// Set is part of the pflag.Value interface. | ||
func (v *KeyspaceTypeFlag) Set(arg string) error { | ||
kt, err := topoproto.ParseKeyspaceType(arg) | ||
if err != nil { | ||
return err | ||
} | ||
|
||
*v = KeyspaceTypeFlag(kt) | ||
|
||
return nil | ||
} | ||
|
||
// String is part of the pflag.Value interface. | ||
func (v *KeyspaceTypeFlag) String() string { | ||
return topoproto.KeyspaceTypeString(topodatapb.KeyspaceType(*v)) | ||
} | ||
|
||
// Type is part of the pflag.Value interface. | ||
func (v *KeyspaceTypeFlag) Type() string { | ||
return "cli.KeyspaceTypeFlag" | ||
} |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,43 @@ | ||
/* | ||
Copyright 20201 The Vitess Authors. | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. typo: 20201=>2021 There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. ✅ |
||
|
||
Licensed under the Apache License, Version 2.0 (the "License"); | ||
you may not use this file except in compliance with the License. | ||
You may obtain a copy of the License at | ||
|
||
http://www.apache.org/licenses/LICENSE-2.0 | ||
|
||
Unless required by applicable law or agreed to in writing, software | ||
distributed under the License is distributed on an "AS IS" BASIS, | ||
WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. | ||
See the License for the specific language governing permissions and | ||
limitations under the License. | ||
*/ | ||
|
||
package cli | ||
|
||
import ( | ||
"vitess.io/vitess/go/vt/topo/topoproto" | ||
|
||
vtctldatapb "vitess.io/vitess/go/vt/proto/vtctldata" | ||
) | ||
|
||
// ParseKeyspaceShards takes a list of positional arguments and converts them to | ||
// vtctldatapb.Shard objects. | ||
func ParseKeyspaceShards(args []string) ([]*vtctldatapb.Shard, error) { | ||
shards := make([]*vtctldatapb.Shard, 0, len(args)) | ||
|
||
for _, arg := range args { | ||
keyspace, shard, err := topoproto.ParseKeyspaceShard(arg) | ||
if err != nil { | ||
return nil, err | ||
} | ||
|
||
shards = append(shards, &vtctldatapb.Shard{ | ||
Keyspace: keyspace, | ||
Name: shard, | ||
}) | ||
} | ||
|
||
return shards, nil | ||
} |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,40 @@ | ||
/* | ||
Copyright 2021 The Vitess Authors. | ||
Licensed under the Apache License, Version 2.0 (the "License"); | ||
you may not use this file except in compliance with the License. | ||
You may obtain a copy of the License at | ||
http://www.apache.org/licenses/LICENSE-2.0 | ||
Unless required by applicable law or agreed to in writing, software | ||
distributed under the License is distributed on an "AS IS" BASIS, | ||
WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. | ||
See the License for the specific language governing permissions and | ||
limitations under the License. | ||
*/ | ||
|
||
package cli | ||
|
||
import ( | ||
"vitess.io/vitess/go/vt/topo/topoproto" | ||
|
||
topodatapb "vitess.io/vitess/go/vt/proto/topodata" | ||
) | ||
|
||
// TabletAliasesFromPosArgs takes a list of positional (non-flag) arguments and | ||
// converts them to tablet aliases. | ||
func TabletAliasesFromPosArgs(args []string) ([]*topodatapb.TabletAlias, error) { | ||
aliases := make([]*topodatapb.TabletAlias, 0, len(args)) | ||
|
||
for _, arg := range args { | ||
alias, err := topoproto.ParseTabletAlias(arg) | ||
if err != nil { | ||
return nil, err | ||
} | ||
|
||
aliases = append(aliases, alias) | ||
} | ||
|
||
return aliases, nil | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
typo: should be augments
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
✅