Skip to content
This repository was archived by the owner on Jun 11, 2025. It is now read-only.

🏷️ Fixed some types to make more clear#10

Merged
nxtcoder17 merged 3 commits into
kloudlite:release-1.0.5from
abdheshnayak:feature/node-job
Jun 23, 2023
Merged

🏷️ Fixed some types to make more clear#10
nxtcoder17 merged 3 commits into
kloudlite:release-1.0.5from
abdheshnayak:feature/node-job

Conversation

@abdheshnayak
Copy link
Copy Markdown
Contributor

updated some types inside nodepool of operator clusters for the awsnodeconfig.

updated some types inside nodepool of operator clusters for the awsnodeconfig.
Comment thread apis/clusters/v1/nodepool_types.go Outdated

// aws -> CloudProvider
NodeConfig string `json:"nodeConfig"`
// NodeConfig string `json:"nodeConfig"`
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

delete this off

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

deleted these comments

Comment thread operators/clusters/internal/controllers/node/node-utils.go Outdated
By Mistake there was NadeName, so fixed it to NodeName and made changes accordingly.
there was some extra comments which are no more needed so deleted those comments.
@nxtcoder17 nxtcoder17 merged commit 4a2c3b2 into kloudlite:release-1.0.5 Jun 23, 2023
@nxtcoder17 nxtcoder17 deleted the feature/node-job branch June 23, 2023 11:40
abdheshnayak pushed a commit that referenced this pull request Nov 5, 2024
router spec updated, things kind of work
abdheshnayak added a commit that referenced this pull request Nov 5, 2024
* 🏷️ Fixed some types to make more clear

updated some types inside nodepool of operator clusters for the awsnodeconfig.

* ✏️ Fixed typos: NadeName -> NodeName in clusters/v1

By Mistake there was NadeName, so fixed it to NodeName and made changes accordingly.

* 💡 Removed some unwanted comments

there was some extra comments which are no more needed so deleted those comments.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants