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 bugzilla component to OWNERS #172
Conversation
/cherry-pick release-4.6 |
@elmiko: once the present PR merges, I will cherry-pick it on top of release-4.6 in a new PR and assign it to you. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
/cherry-pick release-4.4 |
@elmiko: once the present PR merges, I will cherry-pick it on top of release-4.4 in a new PR and assign it to you. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
/cherry-pick release-4.5 |
@elmiko: once the present PR merges, I will cherry-pick it on top of release-4.5 in a new PR and assign it to you. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
Should we also be including the subcomponent on this? |
for this one i think we should not include the subcomponent. subcomponent is optional and the cao could be related in bugzilla to other providers as well (eg ovirt, openstack). |
Technically I think we should have a separate component for CAO, but I would expect any bug that is for the CAO (not the CA) to be related to the cluster-infrastructure team, not any provider team in particular |
i tend to agree about having a separate component, or maybe just a separate subcomponent. imo, considering that "subcomponent" in the OWNERS is optional, we should opt to not use it to ensure we aren't over-scoping. |
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: JoelSpeed 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 |
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.
/lgtm
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.
/lgtm cancel
I don't think the subcomponent is actually optional. BZ usually complains when I try to create a bug without setting it. |
in the docs for changing the OWNERS file it said that "component" is required, but "subcomponent" is optional. i can't link the doc here, but this is the relevant section:
my thinking here is that since CAO could be associated with provider like bare metal or openstack, we should leave the subcomponent empty. |
This change adds the bugzilla component name to the OWNERS file to comply with the prodsec requirements for OpenShift builds through ART.
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.
/lgtm
@elmiko: new pull request created: #173 In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
@elmiko: new pull request created: #174 In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
@elmiko: new pull request created: #175 In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
This change adds the bugzilla component name to the OWNERS file to
comply with the prodsec requirements for OpenShift builds through ART.