Skip to content

Revert "Add and improve logs in ACS bootstrap process (#6890)" - #8229

Closed
weizhouapache wants to merge 1 commit into
apache:mainfrom
weizhouapache:4.19-revert-pr6890
Closed

weizhouapache wants to merge 1 commit into
apache:mainfrom
weizhouapache:4.19-revert-pr6890

Conversation

@weizhouapache

Copy link
Copy Markdown
Member

Description

This PR fixes #8218 by reverting commit 8fa8cef.

Types of changes

  • Breaking change (fix or feature that would cause existing functionality to change)
  • New feature (non-breaking change which adds functionality)
  • Bug fix (non-breaking change which fixes an issue)
  • Enhancement (improves an existing feature and functionality)
  • Cleanup (Code refactoring and cleanup, that may add test cases)
  • build/CI

Feature/Enhancement Scale or Bug Severity

Feature/Enhancement Scale

  • Major
  • Minor

Bug Severity

  • BLOCKER
  • Critical
  • Major
  • Minor
  • Trivial

Screenshots (if appropriate):

How Has This Been Tested?

How did you try to break this feature and the system with this change?

@weizhouapache

Copy link
Copy Markdown
Member Author

@blueorangutan package

@blueorangutan

Copy link
Copy Markdown

@weizhouapache a [SL] Jenkins job has been kicked to build packages. It will be bundled with KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

@GutoVeronezi

Copy link
Copy Markdown
Contributor

@weizhouapache how reverting logs fix the problem?

@codecov

codecov Bot commented Nov 13, 2023

Copy link
Copy Markdown

Codecov Report

Merging #8229 (8b125c7) into main (05b9b6e) will decrease coverage by 18.50%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##               main   #8229       +/-   ##
============================================
- Coverage     22.99%   4.49%   -18.50%     
============================================
  Files          5052     354     -4698     
  Lines        343866   27889   -315977     
  Branches      49530    4873    -44657     
============================================
- Hits          79055    1255    -77800     
+ Misses       253380   26496   -226884     
+ Partials      11431     138    -11293     
Flag Coverage Δ
simulator-marvin-tests ?
uitests 4.49% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

see 4698 files with indirect coverage changes

📣 Codecov offers a browser extension for seamless coverage viewing on GitHub. Try it in Chrome or Firefox today!

@weizhouapache

Copy link
Copy Markdown
Member Author

closed, this may not be needed.

@blueorangutan

Copy link
Copy Markdown

Packaging result [SF]: ✔️ el7 ✔️ el8 ✔️ el9 ✔️ debian ✔️ suse15. SL-JID 7734

@weizhouapache
weizhouapache deleted the 4.19-revert-pr6890 branch December 15, 2023 16: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.

Bug: UI fails with 503 just after a successful install

3 participants