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

feat(ELB): Synchronized and renamed ELB resources and data sources. #1089

Closed
wants to merge 1 commit into from

Conversation

Zippo-Wang
Copy link
Contributor

@Zippo-Wang Zippo-Wang commented Jan 8, 2024

What this PR does / why we need it:

1.Synchrinized resource and data source

  • data.flexibleengine_elb_loadbalancers
  • flexibleengine_lb_member
  • flexibleengine_lb_monitor
  • flexibleengine_lb_whitelist

2.Renamed resource and data resource

  • data.flexibleengine_lb_certificate_v2
  • data.flexibleengine_lb_listeners_v2
  • flexibleengine_lb_l7policy_v2
  • flexibleengine_lb_l7rule_v2
  • flexibleengine_lb_loadbalancer_v2
  • flexibleengine_lb_listener_v2
  • flexibleengine_lb_pool_v2

3.Deprecated

Sort deprecated resource and data source in alphabetical order

Which issue this PR fixes:
(optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged)
fixes #xxx

Special notes for your reviewer:

Release note:


PR Checklist

  • Tests added/passed.
  • Documentation updated.
  • Schema updated.

Acceptance Steps Performed

### data_source_flexibleengine_loadbalancer

=== RUN   TestAccELBV2LoadbalancerDataSource_basic
=== PAUSE TestAccELBV2LoadbalancerDataSource_basic
=== CONT  TestAccELBV2LoadbalancerDataSource_basic
--- PASS: TestAccELBV2LoadbalancerDataSource_basic (188.08s)
PASS

coverage: 2.1% of statements in ../../../terraform-provider-flexibleengine/...
### resource_flexibleengine_member

=== RUN   TestAccLBV2Member_basic
=== PAUSE TestAccLBV2Member_basic
=== CONT  TestAccLBV2Member_basic
--- PASS: TestAccLBV2Member_basic (330.56s)
PASS

coverage: 2.5% of statements in ../../../terraform-provider-flexibleengine/...
### resource_flexibleengine_monitor

=== RUN   TestAccLBV2Monitor_basic
=== PAUSE TestAccLBV2Monitor_basic
=== CONT  TestAccLBV2Monitor_basic
--- PASS: TestAccLBV2Monitor_basic (327.11s)
PASS

coverage: 2.5% of statements in ../../../terraform-provider-flexibleengine/...

=== RUN   TestAccLBV2Monitor_udp
=== PAUSE TestAccLBV2Monitor_udp
=== CONT  TestAccLBV2Monitor_udp
--- PASS: TestAccLBV2Monitor_udp (266.83s)
PASS

coverage: 2.5% of statements in ../../../terraform-provider-flexibleengine/...

=== RUN   TestAccLBV2Monitor_http
=== PAUSE TestAccLBV2Monitor_http
=== CONT  TestAccLBV2Monitor_http
--- PASS: TestAccLBV2Monitor_http (267.60s)
PASS

coverage: 2.5% of statements in ../../../terraform-provider-flexibleengine/...
### resource_flexibleengine_whitelist

=== RUN   TestAccLBV2Whitelist_basic
=== PAUSE TestAccLBV2Whitelist_basic
=== CONT  TestAccLBV2Whitelist_basic
--- PASS: TestAccLBV2Whitelist_basic (260.53s)
PASS

coverage: 2.5% of statements in ../../../terraform-provider-flexibleengine/...

@github-actions github-actions bot added the size/L label Jan 8, 2024
@Zippo-Wang Zippo-Wang changed the title WIP(ELB): Synchronized ELB resources and data sources. WIP: feat(ELB): Synchronized ELB resources and data sources. Jan 8, 2024
@Zippo-Wang Zippo-Wang force-pushed the sync_ELB branch 2 times, most recently from 2d1729d to 9a21637 Compare January 16, 2024 06:41
@Zippo-Wang Zippo-Wang changed the title WIP: feat(ELB): Synchronized ELB resources and data sources. feat(ELB): Synchronized ELB resources and data sources. Jan 16, 2024
@github-actions github-actions bot added size/XL and removed size/L labels Jan 25, 2024
Copy link

This PR exceeds the recommended size of 1000 lines. Please make sure you are NOT addressing multiple issues with one PR. Note this PR might be rejected due to its size.

Copy link

This PR exceeds the recommended size of 1000 lines. Please make sure you are NOT addressing multiple issues with one PR. Note this PR might be rejected due to its size.

Copy link

This PR exceeds the recommended size of 1000 lines. Please make sure you are NOT addressing multiple issues with one PR. Note this PR might be rejected due to its size.

Copy link

This PR exceeds the recommended size of 1000 lines. Please make sure you are NOT addressing multiple issues with one PR. Note this PR might be rejected due to its size.

Copy link

This PR exceeds the recommended size of 1000 lines. Please make sure you are NOT addressing multiple issues with one PR. Note this PR might be rejected due to its size.

Copy link

This PR exceeds the recommended size of 1000 lines. Please make sure you are NOT addressing multiple issues with one PR. Note this PR might be rejected due to its size.

Copy link

This PR exceeds the recommended size of 1000 lines. Please make sure you are NOT addressing multiple issues with one PR. Note this PR might be rejected due to its size.

1 similar comment
Copy link

This PR exceeds the recommended size of 1000 lines. Please make sure you are NOT addressing multiple issues with one PR. Note this PR might be rejected due to its size.

@Zippo-Wang Zippo-Wang changed the title feat(ELB): Synchronized ELB resources and data sources. feat(ELB): Synchronized and renamed ELB resources and data sources. Jan 26, 2024
Copy link

This PR exceeds the recommended size of 1000 lines. Please make sure you are NOT addressing multiple issues with one PR. Note this PR might be rejected due to its size.

2 similar comments
Copy link

This PR exceeds the recommended size of 1000 lines. Please make sure you are NOT addressing multiple issues with one PR. Note this PR might be rejected due to its size.

Copy link

This PR exceeds the recommended size of 1000 lines. Please make sure you are NOT addressing multiple issues with one PR. Note this PR might be rejected due to its size.

Copy link

This PR exceeds the recommended size of 1000 lines. Please make sure you are NOT addressing multiple issues with one PR. Note this PR might be rejected due to its size.

Copy link

This PR exceeds the recommended size of 1000 lines. Please make sure you are NOT addressing multiple issues with one PR. Note this PR might be rejected due to its size.

2 similar comments
Copy link

This PR exceeds the recommended size of 1000 lines. Please make sure you are NOT addressing multiple issues with one PR. Note this PR might be rejected due to its size.

Copy link

This PR exceeds the recommended size of 1000 lines. Please make sure you are NOT addressing multiple issues with one PR. Note this PR might be rejected due to its size.

Copy link

This PR exceeds the recommended size of 1000 lines. Please make sure you are NOT addressing multiple issues with one PR. Note this PR might be rejected due to its size.

@Zippo-Wang Zippo-Wang closed this Mar 4, 2024
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.

1 participant