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): import elb 10sp2 resource and add unit test and docs #1056

Merged
merged 1 commit into from
Dec 25, 2023

Conversation

Zhukun-Huawei
Copy link
Contributor

@Zhukun-Huawei Zhukun-Huawei commented Oct 27, 2023

What this PR does / why we need it:
add lb resources for flexibleengine, including lb_listeners_v2, lb_log_tank_v3 and lb_certificate_v2
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

lb_listeners_v2
=== RUN   TestAccDatasourceListeners_basic
=== PAUSE TestAccDatasourceListeners_basic
=== CONT  TestAccDatasourceListeners_basic
--- PASS: TestAccDatasourceListeners_basic (283.61s)
PASS
coverage: 2.5% of statements in ../../../terraform-provider-flexibleengine/...
lb_log_tank_v3 
=== RUN   TestAccElbLogTank_basic
=== PAUSE TestAccElbLogTank_basic
=== CONT  TestAccElbLogTank_basic
--- PASS: TestAccElbLogTank_basic (343.45s)
PASS
coverage: 2.2% of statements in ../../../terraform-provider-flexibleengine/...
lb_certificate_v2
=== RUN   TestAccDataSourceLBCertificateV2_basic
--- PASS: TestAccDataSourceLBCertificateV2_basic (124.15s)
PASS
coverage: 1.8% of statements in ../../../terraform-provider-flexibleengine/...

@Zippo-Wang
Copy link
Contributor

/lgtm

@github-actions github-actions bot added LGTM and removed LGTM labels Nov 21, 2023
@Zhukun-Huawei Zhukun-Huawei force-pushed the elb_branch branch 2 times, most recently from 54e8a7f to 12501e0 Compare November 24, 2023 09:45
@Zippo-Wang
Copy link
Contributor

/lgtm

@github-actions github-actions bot added the LGTM label Nov 24, 2023
subcategory: "Elastic Load Balance (ELB)"
---

# flexibleengine_lb_listeners
Copy link
Contributor

@chengxiangdong chengxiangdong Nov 25, 2023

Choose a reason for hiding this comment

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

Suggested change
# flexibleengine_lb_listeners
# flexibleengine_lb_listeners_v2

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done, revised

subcategory: "Dedicated Load Balance (Dedicated ELB)"
---

# flexibleengine_elb_logtank
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
# flexibleengine_elb_logtank
# flexibleengine_lb_logtank_v3

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done, revised

subcategory: "Elastic Load Balance (ELB)"
---

# flexibleengine_lb_certificate
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
# flexibleengine_lb_certificate
# flexibleengine_lb_certificate_v2

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done, revised

@github-actions github-actions bot removed the LGTM label Nov 27, 2023
@Zhukun-Huawei Zhukun-Huawei changed the title feat(elb): import elb resource and add unit test and docs feat(elb): import elb 10sp2 resource and add unit test and docs Nov 27, 2023
@Zhukun-Huawei Zhukun-Huawei force-pushed the elb_branch branch 3 times, most recently from 628a94f to bcfc0a1 Compare November 28, 2023 11:52
@chengxiangdong
Copy link
Contributor

/lgtm
/approve

@github-actions github-actions bot added the LGTM label Nov 28, 2023
Copy link
Contributor

Choose a reason for hiding this comment

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

Please change the file name to lb_certificates_v2.md

@Zhukun-Huawei Zhukun-Huawei force-pushed the elb_branch branch 3 times, most recently from 0622576 to c2eeaf8 Compare December 25, 2023 08:40
@ShiChangkuo ShiChangkuo merged commit 1cee08c into FlexibleEngineCloud:master Dec 25, 2023
6 checks passed
@Zhukun-Huawei Zhukun-Huawei deleted the elb_branch branch December 26, 2023 13:12
Zippo-Wang pushed a commit to Zippo-Wang/terraform-provider-flexibleengine that referenced this pull request Jan 3, 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.

4 participants