Bug 1808118: Making logging parameter optional#725
Bug 1808118: Making logging parameter optional#725ricardomaraschini wants to merge 1 commit intoopenshift:masterfrom ricardomaraschini:optional-logging
Conversation
Logging has been deprecated in favour of LogLevel, therefore this field is now optional and should so be updated on api accordingly.
|
@ricardomaraschini: This pull request references Bugzilla bug 1808118, which is valid. The bug has been updated to refer to the pull request using the external bug tracker. 3 validation(s) were run on this bug
DetailsIn 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. |
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: ricardomaraschini The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
|
/assign @dmage Have no idea how I ended loosing this change on the original PR 👎 Could you remember me if this was removed on purpose? Before I assign someone else to review this I would like to have your OK. |
|
/hold |
|
New PR for this #727 |
|
@ricardomaraschini: This pull request references Bugzilla bug 1808118. The bug has been updated to no longer refer to the pull request using the external bug tracker. DetailsIn 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. |
Logging has been deprecated in favour of LogLevel, therefore this field
is now optional and should so be updated on api accordingly.