Skip to content

Conversation

@mtulio
Copy link
Contributor

@mtulio mtulio commented Jul 6, 2022

Suggestion to create tags* on network resources created by CloudFormation templates on UPI installations.
*Tags suggested to be created:

  • All resources: Name=$ClusterName-<resource_name>

Documentation reference that could be also updated:

That change should not impact in the existing flow/installations unless there is any check waiting for untagged resources.

Finally, the Route table ID (Public and Private) will be exported as output to be used (it will be used on CloudFormation templates to create Local Zone subnets).

@openshift-ci openshift-ci bot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Jul 6, 2022
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Jul 6, 2022

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

@openshift-ci openshift-ci bot requested review from jstuever and patrickdillon July 6, 2022 20:56
@mtulio mtulio force-pushed the feat-upi-aws-cfn-vpc branch from 080a0b6 to c4bcbb2 Compare July 6, 2022 22:34
@mtulio mtulio force-pushed the feat-upi-aws-cfn-vpc branch 2 times, most recently from 66ff668 to 5ad0d04 Compare August 11, 2022 21:13
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Aug 11, 2022

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please ask for approval from mtulio by writing /assign @mtulio in a comment. For more information see:The Kubernetes Code Review Process.

The full list of commands accepted by this bot can be found 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

@mtulio mtulio force-pushed the feat-upi-aws-cfn-vpc branch from 5ad0d04 to a1baa61 Compare August 11, 2022 21:26
@mtulio mtulio marked this pull request as ready for review August 11, 2022 21:27
@openshift-ci openshift-ci bot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Aug 11, 2022
@mtulio
Copy link
Contributor Author

mtulio commented Aug 11, 2022

/cc @patrickdillon ptal?

Create tag:Name for all resources created by VPC Stack.
The resource ID PublicRouteTableId and PrivateRouteTableId are also
being exported to be used in any external automation that may need
this information - example: creating new stacks for other purposes
like subnets in edge locations (Local Zones, etc)
@mtulio mtulio force-pushed the feat-upi-aws-cfn-vpc branch from a1baa61 to ce2a44d Compare August 31, 2022 17:47
@mtulio mtulio changed the title chore(upi-aws): Update CloudFormation VPC stack template to set tags on resources and export Rtbs chore(upi-aws): Update CloudFormation VPC stack template to set tags on resources and export public route table Aug 31, 2022
@mtulio
Copy link
Contributor Author

mtulio commented Aug 31, 2022

/retest ci/prow/yaml-lint

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Aug 31, 2022

@mtulio: The /retest command does not accept any targets.
The following commands are available to trigger required jobs:

  • /test aro-unit
  • /test e2e-aws
  • /test e2e-aws-upi
  • /test e2e-azure
  • /test e2e-azure-upi
  • /test e2e-gcp
  • /test e2e-gcp-upi
  • /test e2e-metal-ipi-ovn-ipv6
  • /test e2e-vsphere
  • /test e2e-vsphere-upi
  • /test gofmt
  • /test golint
  • /test govet
  • /test images
  • /test okd-images
  • /test okd-unit
  • /test okd-verify-codegen
  • /test openstack-manifests
  • /test shellcheck
  • /test tf-lint
  • /test unit
  • /test verify-codegen
  • /test verify-vendor
  • /test yaml-lint

The following commands are available to trigger optional jobs:

  • /test e2e-alibaba
  • /test e2e-aws-disruptive
  • /test e2e-aws-fips
  • /test e2e-aws-imdsv2
  • /test e2e-aws-proxy
  • /test e2e-aws-shared-vpc
  • /test e2e-aws-single-node
  • /test e2e-aws-upgrade
  • /test e2e-aws-upi-proxy
  • /test e2e-aws-workers-rhel8
  • /test e2e-azure-resourcegroup
  • /test e2e-azure-shared-vpc
  • /test e2e-azurestack
  • /test e2e-azurestack-upi
  • /test e2e-crc
  • /test e2e-gcp-shared-vpc
  • /test e2e-gcp-upgrade
  • /test e2e-gcp-upi-xpn
  • /test e2e-ibmcloud
  • /test e2e-libvirt
  • /test e2e-metal-assisted
  • /test e2e-metal-ipi
  • /test e2e-metal-ipi-ovn-dualstack
  • /test e2e-metal-ipi-swapped-hosts
  • /test e2e-metal-ipi-virtualmedia
  • /test e2e-metal-single-node-live-iso
  • /test e2e-nutanix
  • /test e2e-openstack
  • /test e2e-openstack-kuryr
  • /test e2e-openstack-parallel
  • /test e2e-openstack-proxy
  • /test e2e-openstack-upi
  • /test e2e-ovirt
  • /test e2e-vsphere-zones
  • /test okd-e2e-aws-ovn
  • /test okd-e2e-aws-upgrade
  • /test okd-e2e-gcp
  • /test okd-e2e-gcp-upgrade
  • /test okd-e2e-vsphere
  • /test tf-fmt

Use /test all to run the following jobs that were automatically triggered:

  • pull-ci-openshift-installer-master-aro-unit
  • pull-ci-openshift-installer-master-e2e-aws-fips
  • pull-ci-openshift-installer-master-e2e-aws-imdsv2
  • pull-ci-openshift-installer-master-e2e-aws-proxy
  • pull-ci-openshift-installer-master-e2e-aws-shared-vpc
  • pull-ci-openshift-installer-master-e2e-aws-single-node
  • pull-ci-openshift-installer-master-e2e-aws-upi
  • pull-ci-openshift-installer-master-e2e-metal-assisted
  • pull-ci-openshift-installer-master-gofmt
  • pull-ci-openshift-installer-master-golint
  • pull-ci-openshift-installer-master-govet
  • pull-ci-openshift-installer-master-images
  • pull-ci-openshift-installer-master-okd-e2e-aws-ovn
  • pull-ci-openshift-installer-master-okd-images
  • pull-ci-openshift-installer-master-okd-unit
  • pull-ci-openshift-installer-master-okd-verify-codegen
  • pull-ci-openshift-installer-master-shellcheck
  • pull-ci-openshift-installer-master-tf-fmt
  • pull-ci-openshift-installer-master-tf-lint
  • pull-ci-openshift-installer-master-unit
  • pull-ci-openshift-installer-master-verify-codegen
  • pull-ci-openshift-installer-master-verify-vendor
  • pull-ci-openshift-installer-master-yaml-lint
Details

In response to this:

/retest ci/prow/yaml-lint

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.

@mtulio mtulio changed the title chore(upi-aws): Update CloudFormation VPC stack template to set tags on resources and export public route table chore(upi-aws): Add tag:Name to resources created by CloudFormation VPC stack template. Aug 31, 2022
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Oct 6, 2022

@mtulio: 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-gcp-ovn ce2a44d link true /test e2e-gcp-ovn
ci/prow/e2e-vsphere-ovn ce2a44d link true /test e2e-vsphere-ovn
ci/prow/e2e-azure-ovn ce2a44d link true /test e2e-azure-ovn
ci/prow/okd-scos-images ce2a44d link true /test okd-scos-images
ci/prow/okd-scos-unit ce2a44d link true /test okd-scos-unit
ci/prow/okd-scos-verify-codegen ce2a44d link true /test okd-scos-verify-codegen

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant