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

Update Roadmap #5085

Merged
merged 2 commits into from
Sep 10, 2024
Merged

Update Roadmap #5085

merged 2 commits into from
Sep 10, 2024

Conversation

dtzar
Copy link
Contributor

@dtzar dtzar commented Aug 20, 2024

What type of PR is this?

/kind documentation

What this PR does / why we need it:
Updates roadmap for project

Release note:

Update roadmap

@k8s-ci-robot k8s-ci-robot added release-note Denotes a PR that will be considered when it comes time to generate release notes. kind/documentation Categorizes issue or PR as related to documentation. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. size/S Denotes a PR that changes 10-29 lines, ignoring generated files. labels Aug 20, 2024
Copy link

codecov bot commented Aug 20, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 51.17%. Comparing base (12d15fa) to head (aec09df).
Report is 48 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #5085      +/-   ##
==========================================
- Coverage   51.18%   51.17%   -0.01%     
==========================================
  Files         273      273              
  Lines       24611    24611              
==========================================
- Hits        12597    12595       -2     
- Misses      11229    11231       +2     
  Partials      785      785              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@dtzar
Copy link
Contributor Author

dtzar commented Aug 23, 2024

FYI - the lint docs error will go away once #5088 is merged

@dtzar dtzar requested review from jackfrancis and removed request for jsturtevant August 23, 2024 18:23
@mboersma
Copy link
Contributor

the lint docs error will go away once #5088 is merged

Since the lint job is a GitHub action, not a prow job, you’ll have to rebase this manually.

Copy link
Contributor

@jackfrancis jackfrancis left a comment

Choose a reason for hiding this comment

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

These are all sensible changes IMO.

/lgtm

/hold for lazy consensus

Shall we share w/ community at next meeting and then expire lazy consensus end of next week?

@k8s-ci-robot k8s-ci-robot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Aug 26, 2024
@k8s-ci-robot k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Aug 26, 2024
@k8s-ci-robot
Copy link
Contributor

LGTM label has been added.

Git tree hash: f9567e70bb29e39271d0a20d21c5d77a8b477cb6

Copy link
Contributor

@mboersma mboersma left a comment

Choose a reason for hiding this comment

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

/lgtm

@nawazkh
Copy link
Member

nawazkh commented Sep 4, 2024

/lgtm

@dtzar
Copy link
Contributor Author

dtzar commented Sep 10, 2024

I think we've met the lazy consensus period.

@mboersma
Copy link
Contributor

/approve

@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: mboersma

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@k8s-ci-robot k8s-ci-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Sep 10, 2024
@mboersma
Copy link
Contributor

/hold cancel

@k8s-ci-robot k8s-ci-robot removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Sep 10, 2024
@k8s-ci-robot k8s-ci-robot merged commit b4b5087 into kubernetes-sigs:main Sep 10, 2024
18 checks passed
@k8s-ci-robot k8s-ci-robot added this to the v1.17 milestone Sep 10, 2024
@dtzar dtzar deleted the roadmap branch September 10, 2024 17:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. kind/documentation Categorizes issue or PR as related to documentation. lgtm "Looks good to me", indicates that a PR is ready to be merged. release-note Denotes a PR that will be considered when it comes time to generate release notes. size/S Denotes a PR that changes 10-29 lines, ignoring generated files.
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

5 participants