-
Notifications
You must be signed in to change notification settings - Fork 1.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
Add ROADMAP document #218
Add ROADMAP document #218
Conversation
|
||
## CNI Milestones | ||
|
||
### [v0.3.0](https://github.com/containernetworking/cni/milestones/v0.3.0) |
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.
peanut gallery, this seems a bit uselessly vague. how about an estimated timeline and/or at least some high level goals for the milestone releases?
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.
Yep, would be good to have some real content ;-) I was wondering if we shall do a mix of roadmap and changelog, opinions?
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.
I think there's enough in the milestones to extract a few highlights to put here
- Stable SPEC | ||
- Strategy and tooling for backwards compatibility | ||
- Complete test coverage | ||
- Integrate build artefact generation with CI |
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.
s/artefact/artifact
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.
as a non-American, I think this is fine http://grammarist.com/spelling/artefact-artifact/
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.
Ah, all good then. We learn American English here in Norway. Sorry about the noise. O:-)
lgtm for now |
I'm happy for this to be merged but can we agree that this roadmap hasn't had much discussion (that I'm aware of) so we should be afraid to change it future? I worry that once it's committed we're somehow committed to it. |
I for one certainly wouldn't consider it set in stone |
It hasn't had much discussion. I assembled it from different conversations, including the recent meetings but also OOB discussions, mostly with @squaremo. It's a first impression and I'm certainly not saying this is how it has to be. As always, it's a start. @squaremo for the sake of mentioning you, I might as well ask you to take a look and comment on the items. Feel free to merge if there's nothing to be changed for now. |
|
||
### [v0.3.0](https://github.com/containernetworking/cni/milestones/v0.3.0) | ||
|
||
* Further Increase test coverage |
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.
Nitpick: I don't think "Increase" needs a capital letter here.
I sympathise with Tom's concern -- an explicit roadmap represents a commitment (whatever one says in the front matter). However we do need to balance that with taking some responsibility for making progress. Perhaps we can compile the roadmap this way:
These are all present already -- nice! The difference is that 0.4 and 0.5 would be elided (but may be discoverable from github milestones, which are understood to be provisional). WDYT @steveej @tomdee ? |
I have been trying to keep issues/PRs to an undefined but limited count to allow a more frequent release cycle, represented in the ROADMAP as well as the issue management. |
|
||
### [v0.4.0](https://github.com/containernetworking/cni/milestones/v0.4.0) | ||
|
||
* Further Increase test coverage |
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.
No capital needed for "increase"
Thanks @steveej and commenters! |
Contributes to #125.
/cc @squaremo