Closed npinaeva closed 8 months ago
Name | Link |
---|---|
Latest commit | 310e46a4e64fac313d5a09b331dd5ddbb9f6dbc4 |
Latest deploy log | https://app.netlify.com/sites/kubernetes-sigs-network-policy-api/deploys/654959808c3764000881fb9c |
Deploy Preview | https://deploy-preview-123--kubernetes-sigs-network-policy-api.netlify.app/npeps/npep-122 |
Preview on mobile | Toggle QR Code...Use your smartphone camera to open QR code link. |
To edit notification comments on pull requests, go to your Netlify site configuration.
Hi @npinaeva. Thanks for your PR.
I'm waiting for a kubernetes-sigs member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test
on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
Once the patch is verified, the new status will be reflected by the ok-to-test
label.
I understand the commands that are listed here.
/ok-to-test
As discussed in the upstream meeting, this version is focused on user stories now, which led me to writing down even more user stories, and also covering existing API fields that I couldn't figure out user stories for. Since the main problem of the existing API is covering much more configs that we actually have user stories for, we probably shouldn't include fields that we don't have user stories for in the next Tenancy API.
/cc
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: astoycos, npinaeva
The full list of commands accepted by this bot can be found here.
The pull request process is described here
/lgtm
/hold cancel
/lgtm
🍾
Change Tenancy implementation from SameLabels/NotSameLabels ANP peers to a separate object that will define Tenancy considering subject and peers are related.
Tracking issue https://github.com/kubernetes-sigs/network-policy-api/issues/122