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

Add initial roadmap #176

Merged
merged 1 commit into from
Apr 19, 2022
Merged

Add initial roadmap #176

merged 1 commit into from
Apr 19, 2022

Conversation

oyvindio
Copy link
Member

This is a suggestion for a roadmap consisting of some items we've discussed internally. Consider this not written in stone; I'm adding it here for transparency's sake, and as a tool for further discussion.

@oyvindio oyvindio requested a review from a team as a code owner March 17, 2022 07:39
Copy link
Member

@xamebax xamebax left a comment

Choose a reason for hiding this comment

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

I wholeheartedly approve this message.

- [S] Evaluate container registries for moving container images out of DockerHub
- [M] Support autoscaling on custom metrics for applications
- [M] Support managing PodDisruptionBudgets for applications
- [L] Look into replacing fiaas/k8s with kubernetes-clients/python
Copy link
Member

Choose a reason for hiding this comment

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

I'd like to hear a bit of background for this item?

Last time I checked, the official client was really cumbersome to work with, especially when dealing with CRDs.
If it was up to me, I would probably prefer to put my time into improving fiaas/k8s in the places where it falls short rather than trying to work with the official client. The only thing I really would like to get from the official client is the config-loading mechanisms.

Copy link
Member

Choose a reason for hiding this comment

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

The background is partially captured by the list of PRs closed in the k8s repository, for example this one: fiaas/k8s#101

We shouldn't be spending time maintaining our own client, because each time an API changes, we have to reflect that. This increases our workload when having to make the FIAAS ecosystem compatible with a new Kubernetes version, and as you know, these come regularly.

Copy link
Member

Choose a reason for hiding this comment

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

Most of those would have been handled with little to no effort if we had taken the time to implement fiaas/k8s#97 (a first attempt of which was in fiaas/k8s#55).

But fair enough. I don't do enough with the fiaas codebase these days to have a strong opinion either way.

Copy link
Member Author

Choose a reason for hiding this comment

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

We shouldn't be spending time maintaining our own client, because each time an API changes, we have to reflect that. This increases our workload when having to make the FIAAS ecosystem compatible with a new Kubernetes version, and as you know, these come regularly.

I think this is a good summary of the background/motivation for this.
Also I see this as initially an exploratory task, to determine if using the official client could be a good fit, before either committing to switching or not.

@oyvindio oyvindio merged commit 6b6d36e into master Apr 19, 2022
@oyvindio oyvindio deleted the add-roadmap branch April 19, 2022 12:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants