Skip to content

Conversation

@r4f4
Copy link
Contributor

@r4f4 r4f4 commented Sep 21, 2022

This fixes the following warning message

DEBUG Warning: Argument is deprecated
DEBUG
DEBUG with aws_s3_bucket_object.ignition,
DEBUG on main.tf line 68, in resource "aws_s3_bucket_object" "ignition":
DEBUG 68: bucket = aws_s3_bucket.ignition.id
DEBUG
DEBUG Use the aws_s3_object resource instead

by using aws_s3_object instead of aws_s3_bucket_object.

This fixes the following warning message

DEBUG Warning: Argument is deprecated
DEBUG
DEBUG   with aws_s3_bucket_object.ignition,
DEBUG   on main.tf line 68, in resource "aws_s3_bucket_object" "ignition":
DEBUG   68:   bucket = aws_s3_bucket.ignition.id
DEBUG
DEBUG Use the aws_s3_object resource instead

by using `aws_s3_object` instead of `aws_s3_bucket_object`.
@r4f4 r4f4 force-pushed the aws-terraform-deprecated-fields branch from 7f85a73 to 8b543d1 Compare September 21, 2022 10:22
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Sep 21, 2022

@r4f4: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
ci/prow/e2e-aws-ovn-single-node 8b543d1 link false /test e2e-aws-ovn-single-node
ci/prow/okd-e2e-gcp-ovn-upgrade 8b543d1 link false /test okd-e2e-gcp-ovn-upgrade
ci/prow/okd-e2e-aws-ovn 8b543d1 link false /test okd-e2e-aws-ovn

Full PR test history. Your PR dashboard.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@jhixson74
Copy link
Member

/lgtm
/approve

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Sep 21, 2022
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Sep 21, 2022

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: jhixson74

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-ci openshift-ci bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Sep 21, 2022
@openshift-merge-robot openshift-merge-robot merged commit 6eca978 into openshift:master Sep 22, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. lgtm Indicates that a PR is ready to be merged.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants