Skip to content

Commit

Permalink
Merge branch 'main' into huijbers/disable-md5-only-in-fips
Browse files Browse the repository at this point in the history
  • Loading branch information
mergify[bot] authored Nov 1, 2024
2 parents fc9d1b7 + 4e715b8 commit 617ffd0
Show file tree
Hide file tree
Showing 11 changed files with 473 additions and 305 deletions.
3 changes: 2 additions & 1 deletion packages/aws-cdk/lib/api/bootstrap/bootstrap-props.ts
Original file line number Diff line number Diff line change
@@ -1,4 +1,5 @@
import { Tag } from '../../cdk-toolkit';
import { StringWithoutPlaceholders } from '../util/placeholders';

export const BUCKET_NAME_OUTPUT = 'BucketName';
export const REPOSITORY_NAME_OUTPUT = 'ImageRepositoryName';
Expand All @@ -17,7 +18,7 @@ export const DEFAULT_BOOTSTRAP_VARIANT = 'AWS CDK: Default Resources';
*/
export interface BootstrapEnvironmentOptions {
readonly toolkitStackName?: string;
readonly roleArn?: string;
readonly roleArn?: StringWithoutPlaceholders;
readonly parameters?: BootstrappingParameters;
readonly force?: boolean;

Expand Down
22 changes: 13 additions & 9 deletions packages/aws-cdk/lib/api/deploy-stack.ts
Original file line number Diff line number Diff line change
Expand Up @@ -19,6 +19,7 @@ import { TemplateBodyParameter, makeBodyParameter } from './util/template-body-p
import { AssetManifestBuilder } from '../util/asset-manifest-builder';
import { determineAllowCrossAccountAssetPublishing } from './util/checks';
import { publishAssets } from '../util/asset-publishing';
import { StringWithoutPlaceholders } from './util/placeholders';

export interface DeployStackResult {
readonly noOp: boolean;
Expand Down Expand Up @@ -51,14 +52,13 @@ export interface DeployStackOptions {
/**
* SDK provider (seeded with default credentials)
*
* Will exclusively be used to assume publishing credentials (which must
* start out from current credentials regardless of whether we've assumed an
* action role to touch the stack or not).
* Will be used to:
*
* Used for the following purposes:
*
* - Publish legacy assets.
* - Upload large CloudFormation templates to the staging bucket.
* - Publish assets, either legacy assets or large CFN templates
* that aren't themselves assets from a manifest. (Needs an SDK
* Provider because the file publishing role is declared as part
* of the asset).
* - Hotswap
*/
readonly sdkProvider: SdkProvider;

Expand All @@ -70,9 +70,13 @@ export interface DeployStackOptions {
/**
* Role to pass to CloudFormation to execute the change set
*
* @default - Role specified on stack, otherwise current
* To obtain a `StringWithoutPlaceholders`, run a regular
* string though `TargetEnvironment.replacePlaceholders`.
*
* @default - No execution role; CloudFormation either uses the role currently associated with
* the stack, or otherwise uses current AWS credentials.
*/
readonly roleArn?: string;
readonly roleArn?: StringWithoutPlaceholders;

/**
* Notification ARNs to pass to CloudFormation to notify when the change set has completed
Expand Down
Loading

0 comments on commit 617ffd0

Please sign in to comment.