Skip to content

Conversation

@davidjumani
Copy link
Contributor

Description

Continuation of #4205

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)

radu-todirica and others added 30 commits October 13, 2022 08:39
Added network offering for tungsten
Creating virtual network in tungsten when we create a network with tungsten offering
Creating virtual machine and virtual machine interface in tungsten when we create a vm with a tungsten network
Added create project/domain in tungsten that match project/domain from cloudstack
Did some code refactoring
Create domain or project in tungsten when a domain or a project is created in cloudstack
Delete domain or project in tungsten when a domain or a project is deleted from cloudstack
- Change setup gw way
- Change vrouter api for multiple host
…tungsten zone is created

Removed the posibility to enable/disable tungsten provider
List tungsten provider info
…ers except Tungsten from the supported services list

when Tungsten checkbox is checked.
-Added unit tests for TungstenGuestNetworkGuru, TungstenApi and TungstenProviderService
-Added integration tests for tungsten network offerings
…ers except Tungsten from the supported services list

when Tungsten checkbox is checked.
-Added unit tests for TungstenGuestNetworkGuru, TungstenApi and TungstenProviderService
-Added integration tests for tungsten network offerings
- Update static nat feature
- Update per virtual router / guest network
- Update tungsten host remove way
- Add introspect api get tungsten link local ip
Exclude the tungsten network dns address from public and management network allocation pool
Exclude tungsten network dns address from the list of available Ip for vms
…ize tungsten db with

cloudstack domains and projects.
- Add get ssl certificate api
@DaanHoogland DaanHoogland removed their assignment Jan 31, 2023
@github-actions
Copy link

This pull request has merge conflicts. Dear author, please fix the conflicts and sync your branch with the base branch.

@davidjumani
Copy link
Contributor Author

@blueorangutan package

@blueorangutan
Copy link

@davidjumani a 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.

Copy link
Contributor

@NuxRo NuxRo left a comment

Choose a reason for hiding this comment

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

LGTM, in its default state (off) Tungsten does not seem to break anything, tested in regular Advanced zones as well as Advanced + Security Groups.

@blueorangutan
Copy link

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

@github-actions
Copy link

This pull request has merge conflicts. Dear author, please fix the conflicts and sync your branch with the base branch.

@davidjumani
Copy link
Contributor Author

@blueorangutan package

@blueorangutan
Copy link

@davidjumani a 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: ✔️ el7 ✔️ el8 ✔️ el9 ✔️ debian ✔️ suse15. SL-JID 5459

@davidjumani
Copy link
Contributor Author

@blueorangutan test matrix

@blueorangutan
Copy link

@davidjumani a Trillian-Jenkins matrix job (centos7 mgmt + xenserver71, rocky8 mgmt + vmware67u3, centos7 mgmt + kvmcentos7) has been kicked to run smoke tests

@sonarqubecloud
Copy link

SonarCloud Quality Gate failed.    Quality Gate failed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 54 Code Smells

21.6% 21.6% Coverage
5.1% 5.1% Duplication

@DaanHoogland DaanHoogland marked this pull request as ready for review January 31, 2023 15:02
@blueorangutan
Copy link

Trillian test result (tid-6048)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 43937 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr7065-t6048-kvm-centos7.zip
Smoke tests completed. 107 look OK, 0 have errors, 0 did not run
Only failed and skipped tests results shown below:

Test Result Time (s) Test File

@blueorangutan
Copy link

Trillian test result (tid-6047)
Environment: vmware-67u3 (x2), Advanced Networking with Mgmt server r8
Total time taken: 44575 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr7065-t6047-vmware-67u3.zip
Smoke tests completed. 107 look OK, 0 have errors, 0 did not run
Only failed and skipped tests results shown below:

Test Result Time (s) Test File

@blueorangutan
Copy link

Trillian test result (tid-6046)
Environment: xenserver-71 (x2), Advanced Networking with Mgmt server 7
Total time taken: 45175 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr7065-t6046-xenserver-71.zip
Smoke tests completed. 107 look OK, 0 have errors, 0 did not run
Only failed and skipped tests results shown below:

Test Result Time (s) Test File

@DaanHoogland
Copy link
Contributor

This PR is a followup of several older ones, it has had a lot of reviews and has been finally implemented as aan experimental feature. It has been manually tested. I deem this safe and desireable, so merging.

@DaanHoogland DaanHoogland merged commit c774b86 into main Feb 1, 2023
@davidjumani davidjumani deleted the tungsten-integration branch February 1, 2023 09:42
@anhtt96VN
Copy link

@huyle-uni Hi anh huyle em là Tuấn Anh kỹ sư IT Việt Nam, em cũng rất quan tâm về Tungsten nhưng còn nhiều thắc mắc không biết em có thể liện hệ với anh để xin chia sẻ về Tungsten từ anh được không ạ?

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.

9 participants