Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Eirene #189

Merged
merged 3 commits into from
Jan 30, 2024
Merged

Eirene #189

merged 3 commits into from
Jan 30, 2024

Conversation

mira-miracoli
Copy link
Contributor

PR for triggering Jenkins, original was

@usegalaxy-eu-bot
Copy link
Collaborator

@hechth
Copy link
Contributor

hechth commented Jan 30, 2024

@bgruening can this be merged?

@mira-miracoli
Copy link
Contributor Author

There was an error in our CI/CD pipeline, I triggered it again, but should pass I think

@usegalaxy-eu-bot
Copy link
Collaborator


No changes. Your infrastructure matches the configuration.

Your configuration already matches the changes detected above. If you'd like
to update the Terraform state to match, create and apply a refresh-only plan:
  terraform apply -refresh-only ```

@hechth
Copy link
Contributor

hechth commented Jan 30, 2024

Thanks a lot - so as far as I understand, the subdomain should be available now? Because I can't see it yet.

@mira-miracoli
Copy link
Contributor Author

There was a merge conflict because we updated another subdomain in the meantime. the CI is running again. sorry

@usegalaxy-eu-bot
Copy link
Collaborator


Terraform used the selected providers to generate the following execution
plan. Resource actions are indicated with the following symbols:
  + create

Terraform will perform the following actions:

  # aws_route53_record.it-subdomain-main[25] will be created
  + resource "aws_route53_record" "it-subdomain-main" {
      + allow_overwrite = true
      + fqdn            = (known after apply)
      + id              = (known after apply)
      + name            = "*.interactivetoolentrypoint.interactivetool.eirene.usegalaxy.eu"
      + records         = [
          + "usegalaxy.eu",
        ]
      + ttl             = 7200
      + type            = "CNAME"
      + zone_id         = "Z1C7L7XFF9613J"
    }

  # aws_route53_record.subdomains[39] will be created
  + resource "aws_route53_record" "subdomains" {
      + allow_overwrite = true
      + fqdn            = (known after apply)
      + id              = (known after apply)
      + name            = "eirene.usegalaxy.eu"
      + records         = [
          + "usegalaxy.eu",
        ]
      + ttl             = 7200
      + type            = "CNAME"
      + zone_id         = "Z1C7L7XFF9613J"
    }

Plan: 2 to add, 0 to change, 0 to destroy.

─────────────────────────────────────────────────────────────────────────────

Saved the plan to: tf.plan

To perform exactly these actions, run the following command to apply:
    terraform apply "tf.plan" ```

@mira-miracoli
Copy link
Contributor Author

I think this is what we want :)

@mira-miracoli mira-miracoli merged commit 61b6c77 into main Jan 30, 2024
3 checks passed
@hechth
Copy link
Contributor

hechth commented Jan 30, 2024

Thanks a lot :)

@bgruening bgruening deleted the eirene branch January 30, 2024 15:07
@bgruening
Copy link
Member

Thanks!

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.

4 participants