feat: Updating Service Token Resource for network based token

[srushti-patl]

• Updated Service Token Resource Schema and Model
• Updated docs for Service Token to add zside Network based token

[ryli17]

With recent change (for Feb release) https://github.com/equinix-product/interconnection_fabric-api-model/pull/1000#pullrequestreview-2518132243, the connection.type is not mandatory anymore. Should we revise models.go accordingly?

[thogarty]

Large PR; I've added the notes for what I was able to see in the first pass. Will review again once we've resolved open discussions.

Please respond to Ryan Li's comment on this PR as well. Could be useful to include that change here as well.

[codecov-commenter]

⚠️ Please install the to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

Attention: Patch coverage is 15.03268% with 130 lines in your changes missing coverage. Please review.

Project coverage is 32.17%. Comparing base (c71308e) to head (53091c3).
Report is 10 commits behind head on main.

Files with missing lines	Patch %	Lines
internal/resources/fabric/service_token/models.go	19.13%	83 Missing and 10 partials ⚠️
...nternal/resources/fabric/service_token/resource.go	0.00%	37 Missing ⚠️

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

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #833      +/-   ##
==========================================
- Coverage   36.13%   32.17%   -3.96%     
==========================================
  Files         184      184              
  Lines       24491    24606     +115     
==========================================
- Hits         8850     7918     -932     
- Misses      15480    16624    +1144     
+ Partials      161       64      -97     

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

[srushti-patl]

With recent change (for Feb release) equinix-product/interconnection_fabric-api-model#1000 (review), the connection.type is not mandatory anymore. Should we revise models.go accordingly?

Confirmed with Ryan that we will change the connection type parameter to optional after the February release. Those changes are not in production right now.

[thogarty]

One small change. The mapping method is critical because it's what's being set to state. It's a good hub to check whenever the plan is showing differences. Very closely related to attribute behaviors.

[thogarty]

LGTM. Ship it.

[github-actions]

This PR is included in version 3.1.0 🎉