-
Notifications
You must be signed in to change notification settings - Fork 539
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Merge pull request #775 from kubernetes-client/master
Sync release-0.x branch to HEAD.
- Loading branch information
Showing
20 changed files
with
408 additions
and
106 deletions.
There are no files selected for viewing
This file contains 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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,73 @@ | ||
# Fetch migration | ||
|
||
Request is fully deprecated requiring us to switch libraries (see [#414](https://github.com/kubernetes-client/javascript/issues/414) for more information). There were a few [different options](https://github.com/kubernetes-client/javascript/issues/414#issuecomment-978031677) for how this swap should be implemented but moving to a new open-api generator option was chosen since this project will acquire the advantages of an up-to-date open-api generator version. | ||
|
||
Fetch was selected as the new HTTP request library for this project due to its widespread adoption across the JavaScript ecosystem. Additonally, potential future updates to this project could allow this client to be available with browser JavaScript since fetch is native to the browser ([#165](https://github.com/kubernetes-client/javascript/issues/165)). | ||
|
||
[Node-fetch](https://www.npmjs.com/package/node-fetch) is our specific fetch package since it is the largest Node.js compatable implementation. Fetch is not implemented by default in Node. | ||
|
||
For more details see the initial discussion ([#754](https://github.com/kubernetes-client/javascript/issues/754)). | ||
|
||
## Release cycle | ||
|
||
The fetch generator will create breaking changes to this project's API. Consumers will have to make small modifications to their code to upgrade. | ||
|
||
We will continue to support the request version of this project for three Kubernetes API versions (~9 months) to give users time to migrate. | ||
|
||
Versioning will follow [npm semantic versioning](https://docs.npmjs.com/about-semantic-versioning). | ||
|
||
### Old generator (request) | ||
|
||
Code will be on the `release-0.x` branch. | ||
|
||
- `0.17.x` == old generator, Kubernetes 1.23 API | ||
- `0.18.x` == old generator, Kubernetes 1.24 API | ||
- `0.19.x` == old generator, Kubernetes 1.25 API | ||
|
||
Support for old generator stops after 1.25 | ||
|
||
### New generator (fetch) | ||
|
||
Code will be on the `master` branch. | ||
|
||
- `1.0.x` == new generator, Kubernetes 1.23 API | ||
- `1.1.x` == new generator, Kubernetes 1.24 API | ||
- `1.2.x` == new generator, Kubernetes 1.25 API | ||
Support for subsequent kubernetes versions continues with the new generator. | ||
|
||
## Implementation steps | ||
|
||
### Other repositories | ||
|
||
- [x] Update [kubernetes-client/gen](https://github.com/kubernetes-client/gen)'s typescript-fetch files to let us pass in the `typescriptThreePlus` config option <sup>[1](https://github.com/OpenAPITools/openapi-generator/issues/9973) [2](https://github.com/OpenAPITools/openapi-generator/issues/3869#issuecomment-584152932)</sub> | ||
- [ ] Update [openapi-generator](https://github.com/OpenAPITools/openapi-generator)'s typescript-fetch flavor to mark parameters as optional if all parameters are optional <sup>[3](https://github.com/OpenAPITools/openapi-generator/issues/6440)</sup> | ||
|
||
### Kubernetes-client repository | ||
|
||
- [ ] Increment `OPENAPI_GENERATOR_COMMIT` to be [version 5.3.0](https://github.com/OpenAPITools/openapi-generator/releases/tag/v5.3.0) (with the optional parameters addition) | ||
- [ ] `npm install node-fetch` to install node-fetch | ||
- [ ] Switch generate-client script to use typescript-fetch | ||
- [ ] Import and inject node-fetch in `src/api.ts` with the following | ||
|
||
```typescript | ||
import fetch from 'node-fetch'; | ||
|
||
// inject node-fetch | ||
if (!globalThis.fetch) { | ||
// @ts-ignore | ||
globalThis.fetch = fetch; | ||
globalThis.Headers = Headers; | ||
globalThis.Request = Request; | ||
globalThis.Response = Response; | ||
} | ||
``` | ||
|
||
- [ ] Generate api with `npm run generate` | ||
- [ ] Match src/gen/api.ts to new generated layout (it changes slightly) | ||
- [ ] Fix errors in /src folder (due to new api) | ||
- [ ] Fix errors in test (due to new api) | ||
- [ ] Test all features | ||
- [ ] Fix examples (due to new api) | ||
- [ ] Update docs | ||
- [ ] Document breaking changes for users | ||
- [ ] Release initial version (1.0.0) |
This file contains 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
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Oops, something went wrong.
This file contains 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
This file contains 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
This file contains 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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1 +1 @@ | ||
4dbc0536a9330c99ca0874d9b9d577fd581a1820517a27f20b4dbf5af0f6fb75 | ||
a5685a88a8d1b58674feda167bae22437c9580a1664511293db4508c164603d2 |
This file contains 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
This file contains 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
This file contains 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
This file contains 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
This file contains 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
Oops, something went wrong.