Skip to content

Commit

Permalink
Update acronyms
Browse files Browse the repository at this point in the history
  • Loading branch information
gemmahou committed Sep 10, 2024
1 parent 71620f3 commit 459ac2d
Show file tree
Hide file tree
Showing 13 changed files with 15 additions and 17 deletions.
2 changes: 1 addition & 1 deletion apis/compute/v1beta1/computeforwardingrule_types.go
Original file line number Diff line number Diff line change
Expand Up @@ -88,7 +88,7 @@ type ForwardingruleServiceDirectoryRegistrations struct {

type ForwardingruleTarget struct {
// +optional
GoogleApiBundle *string `json:"googleApiBundle,omitempty"`
GoogleAPIBundle *string `json:"googleAPIBundle,omitempty"`

// +optional
ServiceAttachmentRef *refs.ComputeServiceAttachmentRef `json:"serviceAttachmentRef,omitempty"`
Expand Down
4 changes: 2 additions & 2 deletions apis/compute/v1beta1/zz_generated.deepcopy.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Original file line number Diff line number Diff line change
Expand Up @@ -473,7 +473,7 @@ spec:
For INTERNAL_SELF_MANAGED load balancing, only HTTP and HTTPS targets
are valid.
properties:
googleApiBundle:
googleAPIBundle:
type: string
serviceAttachmentRef:
oneOf:
Expand Down
1 change: 0 additions & 1 deletion kubebuilder_2.3.1__.tar.gz

This file was deleted.

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion pkg/controller/direct/compute/forwardingrule_controller.go
Original file line number Diff line number Diff line change
Expand Up @@ -286,7 +286,7 @@ func (a *forwardingRuleAdapter) Create(ctx context.Context, createOp *directbase
target := direct.ValueOf(forwardingRule.Target)

// API restriction: Labels are invalid in Private Service Connect Forwarding Rule.
// TF workaround: https://github.com/GoogleCloudPlatform/k8s-config-connector/pull/944
// Config Connector workaround on TF-based resource: https://github.com/GoogleCloudPlatform/k8s-config-connector/pull/944
if target != "all-apis" && target != "vpc-sc" && !strings.Contains(target, "/serviceAttachments/") {
forwardingRule.Labels = desired.Labels
}
Expand Down
4 changes: 2 additions & 2 deletions pkg/controller/direct/compute/mapper.go
Original file line number Diff line number Diff line change
Expand Up @@ -115,8 +115,8 @@ func ComputeForwardingRuleSpec_Target_ToProto(mapCtx *direct.MapContext, in *krm
}

var out *string
if oneof := in.GoogleApiBundle; oneof != nil {
out = in.GoogleApiBundle
if oneof := in.GoogleAPIBundle; oneof != nil {
out = in.GoogleAPIBundle
}
if oneof := in.ServiceAttachmentRef; oneof != nil {
if oneof.External == "" {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@ spec:
networkRef:
name: default
target:
googleApiBundle: all-apis
googleAPIBundle: all-apis
status:
conditions:
- lastTransitionTime: "1970-01-01T00:00:00Z"
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@ spec:
location: global
target:
# a supported Google API bundle (global-only)
googleApiBundle: "all-apis"
googleAPIBundle: "all-apis"
loadBalancingScheme: ""
ipAddress:
addressRef:
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@ spec:
location: global
target:
# a supported Google API bundle (global-only)
googleApiBundle: "all-apis"
googleAPIBundle: "all-apis"
loadBalancingScheme: ""
ipAddress:
addressRef:
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -131,7 +131,7 @@ subnetworkRef:
name: string
namespace: string
target:
googleApiBundle: string
googleAPIBundle: string
serviceAttachmentRef:
external: string
name: string
Expand Down Expand Up @@ -810,7 +810,7 @@ subnetwork must be specified.{% endverbatim %}</p>
</tr>
<tr>
<td>
<p><code>target.googleApiBundle</code></p>
<p><code>target.googleAPIBundle</code></p>
<p><i>Optional</i></p>
</td>
<td>
Expand Down
1 change: 0 additions & 1 deletion tests/apichecks/testdata/exceptions/acronyms.txt
Original file line number Diff line number Diff line change
Expand Up @@ -114,7 +114,6 @@
[acronyms] crd=computefirewallpolicies.compute.cnrm.cloud.google.com version=v1beta1: field ".status.selfLinkWithId" should be ".status.selfLinkWithID"
[acronyms] crd=computeforwardingrules.compute.cnrm.cloud.google.com version=v1beta1: field ".spec.sourceIpRanges" should be ".spec.sourceIPRanges"
[acronyms] crd=computeforwardingrules.compute.cnrm.cloud.google.com version=v1beta1: field ".spec.sourceIpRanges[]" should be ".spec.sourceIPRanges[]"
[acronyms] crd=computeforwardingrules.compute.cnrm.cloud.google.com version=v1beta1: field ".spec.target.googleApiBundle" should be ".spec.target.googleAPIBundle"
[acronyms] crd=computeforwardingrules.compute.cnrm.cloud.google.com version=v1beta1: field ".status.pscConnectionId" should be ".status.pscConnectionID"
[acronyms] crd=computeimages.compute.cnrm.cloud.google.com version=v1beta1: field ".spec.guestOsFeatures" should be ".spec.guestOSFeatures"
[acronyms] crd=computeimages.compute.cnrm.cloud.google.com version=v1beta1: field ".spec.guestOsFeatures[]" should be ".spec.guestOSFeatures[]"
Expand Down

0 comments on commit 459ac2d

Please sign in to comment.