@@ -79,8 +79,8 @@ func TestConnector(t *testing.T) {
7979 subnetRoutes : "10.40.0.0/14" ,
8080 app : kubetypes .AppConnector ,
8181 }
82- expectEqual (t , fc , expectedSecret (t , fc , opts ), nil )
83- expectEqual (t , fc , expectedSTS (t , fc , opts ), removeHashAnnotation )
82+ expectEqual (t , fc , expectedSecret (t , fc , opts ))
83+ expectEqual (t , fc , expectedSTS (t , fc , opts ), removeHashAnnotation , removeResourceReqs )
8484
8585 // Connector status should get updated with the IP/hostname info when available.
8686 const hostname = "foo.tailnetxyz.ts.net"
@@ -106,23 +106,23 @@ func TestConnector(t *testing.T) {
106106 opts .subnetRoutes = "10.40.0.0/14,10.44.0.0/20"
107107 expectReconciled (t , cr , "" , "test" )
108108
109- expectEqual (t , fc , expectedSTS (t , fc , opts ), removeHashAnnotation )
109+ expectEqual (t , fc , expectedSTS (t , fc , opts ), removeHashAnnotation , removeResourceReqs )
110110
111111 // Remove a route.
112112 mustUpdate [tsapi.Connector ](t , fc , "" , "test" , func (conn * tsapi.Connector ) {
113113 conn .Spec .SubnetRouter .AdvertiseRoutes = []tsapi.Route {"10.44.0.0/20" }
114114 })
115115 opts .subnetRoutes = "10.44.0.0/20"
116116 expectReconciled (t , cr , "" , "test" )
117- expectEqual (t , fc , expectedSTS (t , fc , opts ), removeHashAnnotation )
117+ expectEqual (t , fc , expectedSTS (t , fc , opts ), removeHashAnnotation , removeResourceReqs )
118118
119119 // Remove the subnet router.
120120 mustUpdate [tsapi.Connector ](t , fc , "" , "test" , func (conn * tsapi.Connector ) {
121121 conn .Spec .SubnetRouter = nil
122122 })
123123 opts .subnetRoutes = ""
124124 expectReconciled (t , cr , "" , "test" )
125- expectEqual (t , fc , expectedSTS (t , fc , opts ), removeHashAnnotation )
125+ expectEqual (t , fc , expectedSTS (t , fc , opts ), removeHashAnnotation , removeResourceReqs )
126126
127127 // Re-add the subnet router.
128128 mustUpdate [tsapi.Connector ](t , fc , "" , "test" , func (conn * tsapi.Connector ) {
@@ -132,7 +132,7 @@ func TestConnector(t *testing.T) {
132132 })
133133 opts .subnetRoutes = "10.44.0.0/20"
134134 expectReconciled (t , cr , "" , "test" )
135- expectEqual (t , fc , expectedSTS (t , fc , opts ), removeHashAnnotation )
135+ expectEqual (t , fc , expectedSTS (t , fc , opts ), removeHashAnnotation , removeResourceReqs )
136136
137137 // Delete the Connector.
138138 if err = fc .Delete (context .Background (), cn ); err != nil {
@@ -175,16 +175,16 @@ func TestConnector(t *testing.T) {
175175 hostname : "test-connector" ,
176176 app : kubetypes .AppConnector ,
177177 }
178- expectEqual (t , fc , expectedSecret (t , fc , opts ), nil )
179- expectEqual (t , fc , expectedSTS (t , fc , opts ), removeHashAnnotation )
178+ expectEqual (t , fc , expectedSecret (t , fc , opts ))
179+ expectEqual (t , fc , expectedSTS (t , fc , opts ), removeHashAnnotation , removeResourceReqs )
180180
181181 // Add an exit node.
182182 mustUpdate [tsapi.Connector ](t , fc , "" , "test" , func (conn * tsapi.Connector ) {
183183 conn .Spec .ExitNode = true
184184 })
185185 opts .isExitNode = true
186186 expectReconciled (t , cr , "" , "test" )
187- expectEqual (t , fc , expectedSTS (t , fc , opts ), removeHashAnnotation )
187+ expectEqual (t , fc , expectedSTS (t , fc , opts ), removeHashAnnotation , removeResourceReqs )
188188
189189 // Delete the Connector.
190190 if err = fc .Delete (context .Background (), cn ); err != nil {
@@ -261,8 +261,8 @@ func TestConnectorWithProxyClass(t *testing.T) {
261261 subnetRoutes : "10.40.0.0/14" ,
262262 app : kubetypes .AppConnector ,
263263 }
264- expectEqual (t , fc , expectedSecret (t , fc , opts ), nil )
265- expectEqual (t , fc , expectedSTS (t , fc , opts ), removeHashAnnotation )
264+ expectEqual (t , fc , expectedSecret (t , fc , opts ))
265+ expectEqual (t , fc , expectedSTS (t , fc , opts ), removeHashAnnotation , removeResourceReqs )
266266
267267 // 2. Update Connector to specify a ProxyClass. ProxyClass is not yet
268268 // ready, so its configuration is NOT applied to the Connector
@@ -271,7 +271,7 @@ func TestConnectorWithProxyClass(t *testing.T) {
271271 conn .Spec .ProxyClass = "custom-metadata"
272272 })
273273 expectReconciled (t , cr , "" , "test" )
274- expectEqual (t , fc , expectedSTS (t , fc , opts ), removeHashAnnotation )
274+ expectEqual (t , fc , expectedSTS (t , fc , opts ), removeHashAnnotation , removeResourceReqs )
275275
276276 // 3. ProxyClass is set to Ready by proxy-class reconciler. Connector
277277 // get reconciled and configuration from the ProxyClass is applied to
@@ -286,7 +286,7 @@ func TestConnectorWithProxyClass(t *testing.T) {
286286 })
287287 opts .proxyClass = pc .Name
288288 expectReconciled (t , cr , "" , "test" )
289- expectEqual (t , fc , expectedSTS (t , fc , opts ), removeHashAnnotation )
289+ expectEqual (t , fc , expectedSTS (t , fc , opts ), removeHashAnnotation , removeResourceReqs )
290290
291291 // 4. Connector.spec.proxyClass field is unset, Connector gets
292292 // reconciled and configuration from the ProxyClass is removed from the
@@ -296,7 +296,7 @@ func TestConnectorWithProxyClass(t *testing.T) {
296296 })
297297 opts .proxyClass = ""
298298 expectReconciled (t , cr , "" , "test" )
299- expectEqual (t , fc , expectedSTS (t , fc , opts ), removeHashAnnotation )
299+ expectEqual (t , fc , expectedSTS (t , fc , opts ), removeHashAnnotation , removeResourceReqs )
300300}
301301
302302func TestConnectorWithAppConnector (t * testing.T ) {
@@ -351,8 +351,8 @@ func TestConnectorWithAppConnector(t *testing.T) {
351351 app : kubetypes .AppConnector ,
352352 isAppConnector : true ,
353353 }
354- expectEqual (t , fc , expectedSecret (t , fc , opts ), nil )
355- expectEqual (t , fc , expectedSTS (t , fc , opts ), removeHashAnnotation )
354+ expectEqual (t , fc , expectedSecret (t , fc , opts ))
355+ expectEqual (t , fc , expectedSTS (t , fc , opts ), removeHashAnnotation , removeResourceReqs )
356356 // Connector's ready condition should be set to true
357357
358358 cn .ObjectMeta .Finalizers = append (cn .ObjectMeta .Finalizers , "tailscale.com/finalizer" )
@@ -364,7 +364,7 @@ func TestConnectorWithAppConnector(t *testing.T) {
364364 Reason : reasonConnectorCreated ,
365365 Message : reasonConnectorCreated ,
366366 }}
367- expectEqual (t , fc , cn , nil )
367+ expectEqual (t , fc , cn )
368368
369369 // 2. Connector with invalid app connector routes has status set to invalid
370370 mustUpdate [tsapi.Connector ](t , fc , "" , "test" , func (conn * tsapi.Connector ) {
@@ -379,7 +379,7 @@ func TestConnectorWithAppConnector(t *testing.T) {
379379 Reason : reasonConnectorInvalid ,
380380 Message : "Connector is invalid: route 1.2.3.4/5 has non-address bits set; expected 0.0.0.0/5" ,
381381 }}
382- expectEqual (t , fc , cn , nil )
382+ expectEqual (t , fc , cn )
383383
384384 // 3. Connector with valid app connnector routes becomes ready
385385 mustUpdate [tsapi.Connector ](t , fc , "" , "test" , func (conn * tsapi.Connector ) {
0 commit comments