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

refactor: use metal-go for the BGP neighbors data source #368

Merged
merged 1 commit into from
Sep 29, 2023

Conversation

ctreatma
Copy link
Contributor

This updates the device BGP neighbors data source to use metal-go instead of packngo. There were no tests for the BGP neighbors data source, so those are added here as well.

@ctreatma ctreatma changed the title Use metal-go for the BGP neighbors data source refactor: use metal-go for the BGP neighbors data source Aug 31, 2023
@ctreatma ctreatma force-pushed the metal-go-bgp-neighbors branch from 9311470 to aedc757 Compare September 8, 2023 19:57
@ctreatma ctreatma temporarily deployed to internal September 8, 2023 19:57 — with GitHub Actions Inactive
@ctreatma ctreatma force-pushed the metal-go-bgp-neighbors branch from aedc757 to b913082 Compare September 15, 2023 17:07
@ctreatma ctreatma temporarily deployed to internal September 15, 2023 17:07 — with GitHub Actions Inactive
@ctreatma ctreatma requested a deployment to internal September 15, 2023 17:37 — with GitHub Actions Abandoned
@ctreatma ctreatma requested a deployment to internal September 18, 2023 15:04 — with GitHub Actions Abandoned
@equinix equinix deleted a comment from codecov-commenter Sep 18, 2023
@codecov-commenter
Copy link

codecov-commenter commented Sep 18, 2023

Codecov Report

Patch coverage: 77.27% and project coverage change: +46.23% 🎉

Comparison is base (973fdaa) 13.73% compared to head (f30b588) 59.97%.
Report is 5 commits behind head on main.

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@             Coverage Diff             @@
##             main     #368       +/-   ##
===========================================
+ Coverage   13.73%   59.97%   +46.23%     
===========================================
  Files           8       98       +90     
  Lines         910    19753    +18843     
===========================================
+ Hits          125    11847    +11722     
- Misses        770     7605     +6835     
- Partials       15      301      +286     
Files Changed Coverage Δ
equinix/fabric_mapping_helper.go 0.80% <0.00%> (ø)
equinix/data_source_metal_device_bgp_neighbors.go 97.65% <94.11%> (ø)
equinix/fabric_connection_schema.go 100.00% <100.00%> (ø)

... and 87 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@ctreatma ctreatma force-pushed the metal-go-bgp-neighbors branch from b913082 to b69d3b1 Compare September 18, 2023 18:37
@ctreatma ctreatma temporarily deployed to internal September 18, 2023 18:37 — with GitHub Actions Inactive
@ctreatma
Copy link
Contributor Author

This updates the device BGP neighbors data source to use `metal-go`
instead of `packngo`.  There were no tests for the BGP neighbors
data source, so those are added here as well.
@ctreatma ctreatma force-pushed the metal-go-bgp-neighbors branch from b69d3b1 to f30b588 Compare September 21, 2023 16:01
@ctreatma ctreatma temporarily deployed to internal September 21, 2023 16:01 — with GitHub Actions Inactive
@ctreatma ctreatma requested a deployment to internal September 21, 2023 17:07 — with GitHub Actions Abandoned
@ctreatma
Copy link
Contributor Author

@displague displague merged commit 0d5e50a into main Sep 29, 2023
4 of 5 checks passed
@displague displague deleted the metal-go-bgp-neighbors branch September 29, 2023 18:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants