Skip to content

Conversation

@kiranchavala
Copy link
Member

…workOffering

Description

This PR fixes #7510

Types of changes

  • Breaking change (fix or feature that would cause existing functionality to change)
  • New feature (non-breaking change which adds functionality)
  • [ X ] 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)

Feature/Enhancement Scale or Bug Severity

Feature/Enhancement Scale

  • Major
  • [ X ] Minor

Bug Severity

  • BLOCKER
  • Critical
  • Major
  • Minor
  • [ X ] Trivial

Screenshots (if appropriate):

How Has This Been Tested?

@andrijapanicsb
Copy link
Contributor

@blueorangutan LLpackage (testing LL build jobs <----- ignore this)

@blueorangutan
Copy link

@andrijapanicsb a [LL] Jenkins job has been kicked to build packages. It will be bundled with (testing LL build jobs <----- ignore this) SystemVM template(s). I'll keep you posted as I make progress.

@blueorangutan
Copy link

Packaging result [LL]: ✖️ el7 ✖️ el8 ✖️ el9 ✖️ debian ✖️ suse15. SL-JID 6048

@andrijapanicsb
Copy link
Contributor

@blueorangutan package

@blueorangutan
Copy link

@andrijapanicsb a [SF] 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.

@andrijapanicsb
Copy link
Contributor

@blueorangutan LLpackage

@blueorangutan
Copy link

@andrijapanicsb a [LL] 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.

@blueorangutan
Copy link

Packaging result [LL]: ✖️ el7 ✖️ el8 ✖️ el9 ✖️ debian ✖️ suse15. SL-JID 6049

@blueorangutan
Copy link

Packaging result: ✖️ el7 ✖️ el8 ✖️ el9 ✖️ debian ✖️ suse15. SL-JID 6122

weizhouapache and others added 2 commits August 10, 2023 07:46
…ork/UpdateNetworkOfferingCmd.java

Co-authored-by: dahn <daan.hoogland@gmail.com>
…ork/CreateNetworkOfferingCmd.java

Co-authored-by: dahn <daan.hoogland@gmail.com>
Copy link
Member

@weizhouapache weizhouapache left a comment

Choose a reason for hiding this comment

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

code lgtm

@weizhouapache
Copy link
Member

@blueorangutan package

@blueorangutan
Copy link

@weizhouapache a [SF] 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.

@blueorangutan
Copy link

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

…ork/CreateNetworkOfferingCmd.java


remove line trailing
@weizhouapache
Copy link
Member

@blueorangutan package

@blueorangutan
Copy link

@weizhouapache a [SF] 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.

@blueorangutan
Copy link

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

@codecov
Copy link

codecov bot commented Aug 10, 2023

Codecov Report

Merging #7545 (60877e3) into 4.18 (22871ad) will increase coverage by 13.02%.
Report is 194 commits behind head on 4.18.
The diff coverage is n/a.

❗ Current head 60877e3 differs from pull request most recent head 9cac9c8. Consider uploading reports for the commit 9cac9c8 to get more accurate results

@@             Coverage Diff             @@
##             4.18    #7545       +/-   ##
===========================================
+ Coverage        0   13.02%   +13.02%     
- Complexity      0     9032     +9032     
===========================================
  Files           0     2720     +2720     
  Lines           0   257071   +257071     
  Branches        0    40089    +40089     
===========================================
+ Hits            0    33475    +33475     
- Misses          0   219392   +219392     
- Partials        0     4204     +4204     

see 2720 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
Member

@blueorangutan test rocky8 kvm-rocky8

@weizhouapache weizhouapache requested a review from yadvr August 11, 2023 08:03
@yadvr
Copy link
Member

yadvr commented Aug 22, 2023

ping @kiranchavala can you review outstanding comments, thanks.

@weizhouapache
Copy link
Member

moved to 4.18.2.0

@weizhouapache weizhouapache modified the milestones: 4.18.1.0, 4.18.2.0 Aug 24, 2023
@DaanHoogland
Copy link
Contributor

ping @kiranchavala

@DaanHoogland
Copy link
Contributor

@weizhouapache @kiranchavala , can you agree with my suggestion?

@DaanHoogland
Copy link
Contributor

DaanHoogland commented Nov 9, 2023

only textual changes; mergingĤĤĤĤĤĤĤ build errors :(

@DaanHoogland DaanHoogland reopened this Nov 9, 2023
@DaanHoogland DaanHoogland merged commit 8644480 into apache:4.18 Nov 9, 2023
DaanHoogland added a commit that referenced this pull request Nov 10, 2023
* 4.18:
  updated the api documentation for createNetworkOffering and updateNetworkOffering (#7545)
dhslove pushed a commit to ablecloud-team/ablestack-cloud that referenced this pull request Nov 16, 2023
…workOffering (apache#7545)

Co-authored-by: Kiran Chavala <kiranchavala@Kirans-MacBook-Air.local>
Co-authored-by: Wei Zhou <weizhou@apache.org>
Co-authored-by: dahn <daan.hoogland@gmail.com>
dhslove pushed a commit to ablecloud-team/ablestack-cloud that referenced this pull request Nov 16, 2023
* 4.18:
  updated the api documentation for createNetworkOffering and updateNetworkOffering (apache#7545)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants