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

Warn if error is unknown k8s error in destroying namespace #10328

Merged
merged 1 commit into from
Jun 14, 2019

Conversation

ycliuhw
Copy link
Member

@ycliuhw ycliuhw commented Jun 14, 2019

Description of change

Only warn for unknown k8s error message in destroying namespace method.

QA steps

  • prepare a k8s model;
  • kill controller;

Documentation changes

None

Bug reference

None

@ycliuhw ycliuhw requested a review from wallyworld June 14, 2019 02:22
@ycliuhw
Copy link
Member Author

ycliuhw commented Jun 14, 2019

$$merge$$

@jujubot jujubot merged commit 6c4f52c into juju:2.6 Jun 14, 2019
jujubot added a commit that referenced this pull request Jun 19, 2019
#10352

## Description of change

Merge 2.6 forward to bring in:
- #10340 from hpidcock/1831682-k8s-bootstrap-errors
- #10347 from hpidcock/1832777-enable-ha-panic-fix
- #10332 from hpidcock/1832608-statefulset-servicename
- #10330 from ycliuhw/fix/podspec-cmd-args-parsing
- #10335 from babbageclunk/remove-saas-bug
- #10343 from hmlanigan/schema
- #10337 from SimonRichardson/missing-exit
- #10336 from SimonRichardson/schema-gen
- #10327 from manadart/2.6-cache-entity-copies
- #10328 from ycliuhw/fix/warn-message
- #10320 from hpidcock/1832608-statefulset-servicename
@ycliuhw ycliuhw deleted the fix/warn-message branch June 20, 2019 01:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants