Skip to content
This repository has been archived by the owner on Aug 23, 2024. It is now read-only.

update GitHub actions to use equinix-labs #64

Merged
merged 8 commits into from
Jun 3, 2024
Merged

Conversation

displague
Copy link
Member

Use latest equinix-labs published GitHub actions

@displague
Copy link
Member Author

displague commented Jun 3, 2024

GitHub actions ran successfully and setup and teardown worked great.

However, integration CI did not run cleanly for terraform apply so the test should not have been marked as successful.

This continue_on_error: true should be replaced with if: always() (or if: success() || failure() which is cancelable and could be used to skip the destroy step)


╷
│ Error: remote-exec provisioner error
│ 
│   with null_resource.apply_esx_network_config[1],
│   on main.tf line 397, in resource "null_resource" "apply_esx_network_config":
│  397:   provisioner "remote-exec" {
│ 
│ error executing "/tmp/terraform_1777450914.sh": Process exited with status
│ 1
╵
╷
│ Error: remote-exec provisioner error
│ 
│   with null_resource.apply_esx_network_config[0],
│   on main.tf line 397, in resource "null_resource" "apply_esx_network_config":
│  397:   provisioner "remote-exec" {
│ 
│ error executing "/tmp/terraform_478142771.sh": Process exited with status 1
╵
Error: Terraform exited with code 1.
Error: Process completed with exit code 1.

(This timeout was previously noted here: https://github.com/equinix/terraform-metal-vsphere/pull/55/files#r1188437286 )

Signed-off-by: Marques Johansson <mjohansson@equinix.com>
@displague displague merged commit 28441ee into main Jun 3, 2024
1 of 4 checks passed
@displague displague deleted the displague-patch-1 branch June 3, 2024 15:15
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant