Commit 9a27fef
authored
Update configure_index index for new API (#281)
## Problem
The shape of the configure_index shape has changed, so we need to pass
different stuff.
## Solution
- Construct new request body
- Add minimal integration test. I just want to see the request succeed
for now. The time needed for changes to take effect is variable, and I'm
in a bit of a rush. So I don't want to do a bunch of polling for status
updates in my test to verify the server actually did the correct thing.
- Delete some other legacy tests in `integ/` that have been disabled for
a while now.
## Type of Change
- [x] Bug fix (non-breaking change which fixes an issue)
## Test Plan
New test added.1 parent 44fc7ed commit 9a27fef
File tree
5 files changed
+18
-142
lines changed- pinecone/control
- tests
- integration/control
- integ
5 files changed
+18
-142
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
12 | 12 | | |
13 | 13 | | |
14 | 14 | | |
| 15 | + | |
| 16 | + | |
15 | 17 | | |
16 | 18 | | |
17 | 19 | | |
| |||
176 | 178 | | |
177 | 179 | | |
178 | 180 | | |
179 | | - | |
| 181 | + | |
| 182 | + | |
| 183 | + | |
| 184 | + | |
| 185 | + | |
180 | 186 | | |
181 | 187 | | |
182 | 188 | | |
| |||
Whitespace-only changes.
This file was deleted.
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
35 | 35 | | |
36 | 36 | | |
37 | 37 | | |
38 | | - | |
39 | | - | |
40 | | - | |
| 38 | + | |
| 39 | + | |
| 40 | + | |
| 41 | + | |
| 42 | + | |
| 43 | + | |
41 | 44 | | |
42 | 45 | | |
43 | 46 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
| 1 | + | |
| 2 | + | |
| 3 | + | |
| 4 | + | |
| 5 | + | |
0 commit comments