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

[RA2 Ch06] Add most of sigs in RA2 chapter 6 #2400

Merged
merged 1 commit into from May 27, 2021

Conversation

collivier
Copy link
Collaborator

It remains sig-storage which asks for a deep dive on both features
and drivers. As starting point, the features list conforms with the
upstream K8s gates (kind).

It allows RA2 team to update the mandatory choices which RC2 will
conform. It's worth mentioning that a few related test could be still
skipped in RC2 (only for gce, disruptive, alpha, etc.).

Signed-off-by: Cédric Ollivier cedric.ollivier@orange.com

It remains sig-storage which asks for a deep dive on both features
and drivers. As starting point, the features list conforms with the
upstream K8s gates (kind).

It allows RA2 team to update the mandatory choices which RC2 will
conform. It's worth mentioning that a few related test could be still
skipped in RC2 (only for gce, disruptive, alpha, etc.).

Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
@rgstori rgstori added the Kali Release Name for 1h2021 label May 6, 2021
@rgstori rgstori added this to In Progress in old-RA2 via automation May 6, 2021
@rgstori rgstori added this to the Kali - M3 - Content Freeze milestone May 6, 2021
@rgstori rgstori requested a review from a team May 6, 2021 15:06
@pgoyal01 pgoyal01 changed the title Add most of sigs in RA2 chapter 6 [RA2 Ch06] Add most of sigs in RA2 chapter 6 May 6, 2021
| **Labels** | **Mandatory** |
|------------------------------|:-------------:|
| Conformance | X |
| None | X |
Copy link
Collaborator

Choose a reason for hiding this comment

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

What is None here?

Copy link
Collaborator

Choose a reason for hiding this comment

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

And in all other SIG-s?

Copy link
Collaborator

Choose a reason for hiding this comment

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

these are labels. "None" selects all functest objects that have the label "none".

Copy link
Collaborator

Choose a reason for hiding this comment

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

"Conformance" means it's in the k8s conformance test, and "None" means mainstream features


| **Labels** | **Mandatory** |
|------------------------------|:-------------:|
| Conformance | X |
Copy link
Collaborator

Choose a reason for hiding this comment

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

What is Conformance here?

Copy link
Collaborator

Choose a reason for hiding this comment

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

And in all other SIG-s? :)

Copy link
Collaborator

Choose a reason for hiding this comment

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

these are labels. "Conformance" selects all functest objects that have the label "Conformance".

Copy link
Collaborator

Choose a reason for hiding this comment

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

"Conformance" means it's in the k8s conformance test, and "None" means mainstream features

Copy link
Collaborator

Choose a reason for hiding this comment

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

Can we have an explanation of this in the intro chapter? I'm okay to have this in an other pr.

| None | X |
| Feature:DaemonSetUpdateSurge | |
| Feature:IndexedJob | |
| Feature:StatefulSet | |
Copy link
Collaborator

Choose a reason for hiding this comment

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

I think StatefulSet is needed.

Copy link
Collaborator

Choose a reason for hiding this comment

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

yes, let's add this to a new PR though (including adding columns for the description of each feature in plain English and the traceability to ch4 specs)

Copy link
Collaborator

Choose a reason for hiding this comment

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

Okey. Should I create an issue to capture this?

Copy link
Collaborator

Choose a reason for hiding this comment

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

should be #2440

| Feature:Networking-IPv4 | X |
| Feature:Networking-IPv6 | |
| Feature:Networking-Performance | X |
| Feature:NetworkPolicy | |
Copy link
Collaborator

Choose a reason for hiding this comment

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

I think NetworkPolicy should be supported, however I could not find any supporting requirement in Ch4.

Copy link
Collaborator

Choose a reason for hiding this comment

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

let's add one to chapter 4 first then, then add the X here with a new PR
(or not)

@rgstori rgstori requested a review from a team May 20, 2021 15:35
@rgstori rgstori dismissed CsatariGergely’s stale review May 27, 2021 15:28

to be done in a separate PR

@rgstori rgstori merged commit dd4ffa7 into anuket-project:master May 27, 2021
old-RA2 automation moved this from In Progress to Done May 27, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Kali Release Name for 1h2021
Projects
No open projects
old-RA2
  
Done
Development

Successfully merging this pull request may close these issues.

None yet

5 participants