Skip to content

cleanup processed files in VR - #7632

Closed
DaanHoogland wants to merge 1 commit into
apache:4.18from
shapeblue:logRotateAndSuch
Closed

cleanup processed files in VR#7632
DaanHoogland wants to merge 1 commit into
apache:4.18from
shapeblue:logRotateAndSuch

Conversation

@DaanHoogland

Copy link
Copy Markdown
Contributor

Description

When a PR is long running, the directory /var/cache/cloud/processed filles up and might glock the filesystem, rendring the VR unusable.
This PR implements a mech for cleaning that directory on demand.
(to be triggered yet, by API/schedule/configurable timer???)

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)

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?

@DaanHooglandDaanHoogland changed the title cleanup processed in VRcleanup processed files in VRJun 19, 2023
@apacheapache deleted a comment from blueorangutanJun 19, 2023
@apacheapache deleted a comment from blueorangutanJun 19, 2023
@codecov

codecovBot commented Jun 19, 2023

Copy link
Copy Markdown

Codecov Report

Merging #7632 (38089c7) into 4.18 (e4117e6) will decrease coverage by 0.04%.
The diff coverage is 54.76%.

❗ Current head 38089c7 differs from pull request most recent head 555e054. Consider uploading reports for the commit 555e054 to get more accurate results

@@ Coverage Diff @@## 4.18 #7632 +/- ##
============================================
- Coverage 13.05% 13.02% -0.04% + Complexity 9083 9032 -51 
============================================
Files 2720 2720 Lines 257370 257067 -303 Branches 40124 40087 -37 ============================================
- Hits 33612 33476 -136 + Misses 219537 219387 -150 + Partials 4221 4204 -17 
Files ChangedCoverage Δ
...va/com/cloud/agent/manager/DirectAgentAttache.java9.83% <0.00%> (ø)
...tack/engine/orchestration/NetworkOrchestrator.java6.06% <0.00%> (ø)
...ork/router/VirtualNetworkApplianceManagerImpl.java2.72% <0.00%> (-0.01%)⬇️
...network/router/VirtualNetworkApplianceManager.java100.00% <100.00%> (ø)

... and 29 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@weizhouapacheweizhouapache added this to the 4.18.1.0 milestone Jun 20, 2023
@apacheapache deleted a comment from blueorangutanJul 11, 2023
@apacheapache deleted a comment from blueorangutanJul 11, 2023
@apacheapache deleted a comment from blueorangutanJul 11, 2023
@apacheapache deleted a comment from blueorangutanJul 11, 2023
@apacheapache deleted a comment from blueorangutanJul 11, 2023
@apacheapache deleted a comment from blueorangutanJul 11, 2023

import com.cloud.agent.api.routing.NetworkElementCommand;

public class CleanProcessedCacheCommand extends NetworkElementCommand {

Copy link
Copy Markdown
ContributorAuthor

Choose a reason for hiding this comment

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

nts: does this mean the answer should also inherit (from NetworkElementAnswer for instance)

@apacheapache deleted a comment from blueorangutanJul 18, 2023
@apacheapache deleted a comment from blueorangutanJul 18, 2023
@DaanHoogland
DaanHooglandforce-pushed the logRotateAndSuch branch 2 times, most recently from 4792a6c to 1563e28CompareJuly 26, 2023 07:13
@apacheapache deleted a comment from blueorangutanAug 4, 2023
@apacheapache deleted a comment from blueorangutanAug 4, 2023
@apacheapache deleted a comment from blueorangutanAug 7, 2023
@apacheapache deleted a comment from blueorangutanAug 7, 2023
@apacheapache deleted a comment from blueorangutanAug 7, 2023
@apacheapache deleted a comment from blueorangutanAug 7, 2023
@DaanHoogland

Copy link
Copy Markdown
ContributorAuthor

@blueorangutan package

@blueorangutan

Copy link
Copy Markdown

@DaanHoogland 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
Copy Markdown

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

@DaanHoogland

Copy link
Copy Markdown
ContributorAuthor

@blueorangutan test keepEnv

@blueorangutan

Copy link
Copy Markdown

@DaanHoogland a [SF] Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

@blueorangutan

Copy link
Copy Markdown

[SF] Trillian test result (tid-7316)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 39009 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr7632-t7316-kvm-centos7.zip
Smoke tests completed. 95 look OK, 13 have errors, 0 did not run
Only failed and skipped tests results shown below:

TestResultTime (s)Test File
test_network_aclError39.99test_network_acl.py
test_01_vpc_privategw_aclFailure42.66test_privategw_acl.py
test_02_vpc_privategw_static_routesFailure39.65test_privategw_acl.py
test_03_vpc_privategw_restart_vpc_cleanupFailure42.62test_privategw_acl.py
test_04_rvpc_privategw_static_routesFailure46.12test_privategw_acl.py
test_10_vpc_tier_kubernetes_clusterError82.22test_kubernetes_clusters.py
test_02_native_to_native_vpc_migrationError40.80test_migration.py
test_03_create_vpc_domain_vpc_offeringError51.28test_domain_vpc_offerings.py
test_01_internallb_roundrobin_1VPC_3VM_HTTP_port80Failure46.52test_internal_lb.py
test_02_internallb_roundrobin_1RVPC_3VM_HTTP_port80Failure46.54test_internal_lb.py
test_03_vpc_internallb_haproxy_stats_on_all_interfacesFailure44.63test_internal_lb.py
test_04_rvpc_internallb_haproxy_stats_on_all_interfacesFailure46.49test_internal_lb.py
test_01_single_VPC_iptables_policiesError41.16test_routers_iptables_default_policy.py
test_07_autoscaling_vmgroup_on_vpc_networkError43.05test_vm_autoscaling.py
test_01_migrate_VM_and_root_volumeError81.75test_vm_life_cycle.py
test_02_migrate_VM_with_two_data_disksError53.38test_vm_life_cycle.py
test_01_verify_ipv6_vpcError41.15test_vpc_ipv6.py
test_01_create_redundant_VPC_2tiers_4VMs_4IPs_4PF_ACLError41.79test_vpc_redundant.py
test_02_redundant_VPC_default_routesError42.82test_vpc_redundant.py
test_03_create_redundant_VPC_1tier_2VMs_2IPs_2PF_ACL_reboot_routersError45.82test_vpc_redundant.py
test_04_rvpc_network_garbage_collector_nicsError43.77test_vpc_redundant.py
test_05_rvpc_multi_tiersError41.72test_vpc_redundant.py
test_01_VPC_nics_after_destroyError39.56test_vpc_router_nics.py
test_02_VPC_default_routesError40.68test_vpc_router_nics.py
test_01_redundant_vpc_site2site_vpnFailure42.20test_vpc_vpn.py
test_01_vpc_site2site_vpn_multiple_optionsFailure43.21test_vpc_vpn.py
test_01_vpc_remote_access_vpnFailure34.87test_vpc_vpn.py
test_01_vpc_site2site_vpnFailure44.36test_vpc_vpn.py

@DaanHoogland

Copy link
Copy Markdown
ContributorAuthor

vpc creation failes after these changes: needs work

@weizhouapache

Copy link
Copy Markdown
Member

@DaanHoogland
there was a commit for similar issue see #5331
the script runs once every day. we could let it run every hour or every minute.

@DaanHoogland

Copy link
Copy Markdown
ContributorAuthor

@DaanHoogland there was a commit for similar issue see #5331 the script runs once every day. we could let it run every hour or every minute.

let's push this forward then (marking 4.18.2 for now and we can discuss if it is still applicable)

@DaanHooglandDaanHoogland modified the milestones: 4.18.1.0, 4.18.2.0Aug 23, 2023
@DaanHoogland

Copy link
Copy Markdown
ContributorAuthor

@blueorangutan LLpackage

@blueorangutan

Copy link
Copy Markdown

@DaanHoogland 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
Copy Markdown

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

@DaanHoogland

Copy link
Copy Markdown
ContributorAuthor

@blueorangutan LLtest matrix

@blueorangutan

Copy link
Copy Markdown

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

@blueorangutan

Copy link
Copy Markdown

[LL] Trillian Build Failed (tid-6763)

@DaanHoogland

Copy link
Copy Markdown
ContributorAuthor

no interest

Sign up for freeto 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.

3 participants

@DaanHoogland@blueorangutan@weizhouapache