Skip to content

Commit

Permalink
fix: preserve default values in x-goog-request-params header (#188)
Browse files Browse the repository at this point in the history
- [ ] Regenerate this pull request now.

PiperOrigin-RevId: 474338479

Source-Link: googleapis/googleapis@d5d35e0

Source-Link: googleapis/googleapis-gen@efcd3f9
Copy-Tag: eyJwIjoiLmdpdGh1Yi8uT3dsQm90LnlhbWwiLCJoIjoiZWZjZDNmOTM5NjJhMTAzZjY4ZjAwM2UyYTFlZWNkZTZmYTIxNmEyNyJ9

feat: Add compliant_but_disallowed_services field to the v1beta1 Workload proto
Committer: @samhutton
PiperOrigin-RevId: 471822190

Source-Link: googleapis/googleapis@61f75b4

Source-Link: googleapis/googleapis-gen@9abf7bc
Copy-Tag: eyJwIjoiLmdpdGh1Yi8uT3dsQm90LnlhbWwiLCJoIjoiOWFiZjdiYzI3MTU3MGY3MDczYzg5YzBkNGJlYjNiODZhNjY2NzhjYiJ9
  • Loading branch information
gcf-owl-bot[bot] authored Sep 14, 2022
1 parent 0f475be commit bd2145b
Show file tree
Hide file tree
Showing 10 changed files with 510 additions and 488 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -535,6 +535,12 @@ message Workload {
// SAA enrollment response is queried during GetWorkload call.
// In failure cases, user friendly error message is shown in SAA details page.
SaaEnrollmentResponse saa_enrollment_response = 20 [(google.api.field_behavior) = OUTPUT_ONLY];

// Output only. Urls for services which are compliant for this Assured Workload, but which
// are currently disallowed by the ResourceUsageRestriction org policy.
// Invoke RestrictAllowedResources endpoint to allow your project developers
// to use these services in their environment."
repeated string compliant_but_disallowed_services = 24 [(google.api.field_behavior) = OUTPUT_ONLY];
}

// Operation metadata to give request details of CreateWorkload.
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -87,8 +87,8 @@ service AssuredWorkloadsService {
option (google.api.method_signature) = "name";
}

// A request to analyze a hypothetical move of a source project or
// project-based workload to a target (destination) folder-based workload.
// Analyze if the source Assured Workloads can be moved to the target Assured
// Workload
rpc AnalyzeWorkloadMove(AnalyzeWorkloadMoveRequest) returns (AnalyzeWorkloadMoveResponse) {
option (google.api.method_signature) = "project,target";
}
Expand Down
6 changes: 6 additions & 0 deletions packages/google-cloud-assuredworkloads/protos/protos.d.ts

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

39 changes: 39 additions & 0 deletions packages/google-cloud-assuredworkloads/protos/protos.js

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

8 changes: 8 additions & 0 deletions packages/google-cloud-assuredworkloads/protos/protos.json

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 @@ -235,7 +235,7 @@
"regionTag": "assuredworkloads_v1beta1_generated_AssuredWorkloadsService_AnalyzeWorkloadMove_async",
"title": "AssuredWorkloadsService analyzeWorkloadMove Sample",
"origin": "API_DEFINITION",
"description": " A request to analyze a hypothetical move of a source project or project-based workload to a target (destination) folder-based workload.",
"description": " Analyze if the source Assured Workloads can be moved to the target Assured Workload",
"canonical": true,
"file": "assured_workloads_service.analyze_workload_move.js",
"language": "JAVASCRIPT",
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -482,7 +482,7 @@ export class AssuredWorkloadsServiceClient {
options.otherArgs.headers = options.otherArgs.headers || {};
options.otherArgs.headers['x-goog-request-params'] =
this._gaxModule.routingHeader.fromParams({
'workload.name': request.workload!.name || '',
'workload.name': request.workload!.name ?? '',
});
this.initialize();
return this.innerApiCalls.updateWorkload(request, options, callback);
Expand Down Expand Up @@ -586,7 +586,7 @@ export class AssuredWorkloadsServiceClient {
options.otherArgs.headers = options.otherArgs.headers || {};
options.otherArgs.headers['x-goog-request-params'] =
this._gaxModule.routingHeader.fromParams({
name: request.name || '',
name: request.name ?? '',
});
this.initialize();
return this.innerApiCalls.deleteWorkload(request, options, callback);
Expand Down Expand Up @@ -681,7 +681,7 @@ export class AssuredWorkloadsServiceClient {
options.otherArgs.headers = options.otherArgs.headers || {};
options.otherArgs.headers['x-goog-request-params'] =
this._gaxModule.routingHeader.fromParams({
name: request.name || '',
name: request.name ?? '',
});
this.initialize();
return this.innerApiCalls.getWorkload(request, options, callback);
Expand Down Expand Up @@ -793,7 +793,7 @@ export class AssuredWorkloadsServiceClient {
options.otherArgs.headers = options.otherArgs.headers || {};
options.otherArgs.headers['x-goog-request-params'] =
this._gaxModule.routingHeader.fromParams({
parent: request.parent || '',
parent: request.parent ?? '',
});
this.initialize();
return this.innerApiCalls.createWorkload(request, options, callback);
Expand Down Expand Up @@ -932,7 +932,7 @@ export class AssuredWorkloadsServiceClient {
options.otherArgs.headers = options.otherArgs.headers || {};
options.otherArgs.headers['x-goog-request-params'] =
this._gaxModule.routingHeader.fromParams({
parent: request.parent || '',
parent: request.parent ?? '',
});
this.initialize();
return this.innerApiCalls.listWorkloads(request, options, callback);
Expand Down Expand Up @@ -976,7 +976,7 @@ export class AssuredWorkloadsServiceClient {
options.otherArgs.headers = options.otherArgs.headers || {};
options.otherArgs.headers['x-goog-request-params'] =
this._gaxModule.routingHeader.fromParams({
parent: request.parent || '',
parent: request.parent ?? '',
});
const defaultCallSettings = this._defaults['listWorkloads'];
const callSettings = defaultCallSettings.merge(options);
Expand Down Expand Up @@ -1029,7 +1029,7 @@ export class AssuredWorkloadsServiceClient {
options.otherArgs.headers = options.otherArgs.headers || {};
options.otherArgs.headers['x-goog-request-params'] =
this._gaxModule.routingHeader.fromParams({
parent: request.parent || '',
parent: request.parent ?? '',
});
const defaultCallSettings = this._defaults['listWorkloads'];
const callSettings = defaultCallSettings.merge(options);
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -582,7 +582,7 @@ export class AssuredWorkloadsServiceClient {
options.otherArgs.headers = options.otherArgs.headers || {};
options.otherArgs.headers['x-goog-request-params'] =
this._gaxModule.routingHeader.fromParams({
name: request.name || '',
name: request.name ?? '',
});
this.initialize();
return this.innerApiCalls.restrictAllowedResources(
Expand Down Expand Up @@ -693,7 +693,7 @@ export class AssuredWorkloadsServiceClient {
options.otherArgs.headers = options.otherArgs.headers || {};
options.otherArgs.headers['x-goog-request-params'] =
this._gaxModule.routingHeader.fromParams({
name: request.name || '',
name: request.name ?? '',
});
this.initialize();
return this.innerApiCalls.deleteWorkload(request, options, callback);
Expand Down Expand Up @@ -796,8 +796,8 @@ export class AssuredWorkloadsServiceClient {
return this.innerApiCalls.getWorkload(request, options, callback);
}
/**
* A request to analyze a hypothetical move of a source project or
* project-based workload to a target (destination) folder-based workload.
* Analyze if the source Assured Workloads can be moved to the target Assured
* Workload
*
* @param {Object} request
* The request object that will be sent.
Expand Down Expand Up @@ -1014,7 +1014,7 @@ export class AssuredWorkloadsServiceClient {
options.otherArgs.headers = options.otherArgs.headers || {};
options.otherArgs.headers['x-goog-request-params'] =
this._gaxModule.routingHeader.fromParams({
parent: request.parent || '',
parent: request.parent ?? '',
});
this.initialize();
return this.innerApiCalls.createWorkload(request, options, callback);
Expand Down
Loading

0 comments on commit bd2145b

Please sign in to comment.