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

Check if DNS records exist before posting comment with subdomains #607

Merged
merged 3 commits into from
Mar 14, 2024

Conversation

chainchad
Copy link
Collaborator

Motivation

When setting up a CRIB, we make a PR comment with the subdomains. We don't want to post those subdomains unless they have been propagated.

Solution

Check AWS Route 53 for these records and then post the comment with the subdomain records.

@chainchad chainchad requested review from RensR, connorwstein, jasonmci and a team as code owners March 13, 2024 18:23
@chainchad chainchad added the crib Create ephemeral environment. label Mar 13, 2024
@chainchad chainchad removed the crib Create ephemeral environment. label Mar 13, 2024
@chainchad chainchad added the crib Create ephemeral environment. label Mar 13, 2024
Copy link
Contributor

CRIB Environment Details ℹ️

CRIB activated via the 'crib' label. To destroy the environment, remove the 'crib' PR label or close the PR.

Please review the following details:

Subdomains

Use these subdomains to access the CRIB environment. They are prefixes to the internal base domain which work over the VPN.

  • crib-ccip-607-node1.
  • crib-ccip-607-node2.
  • crib-ccip-607-node3.
  • crib-ccip-607-node4.
  • crib-ccip-607-node5.
  • crib-ccip-607-node6.
  • crib-ccip-607-geth-http.
  • crib-ccip-607-geth-ws.
  • crib-ccip-607-mockserver.

NOTE: If you have trouble resolving these subdomains, please try to reset your VPN DNS and/or local DNS.

@chainchad chainchad removed the crib Create ephemeral environment. label Mar 13, 2024
Copy link
Contributor

@mateusz-sekara mateusz-sekara left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

hey @AnieeG , can you take a look as well?

@chainchad chainchad requested a review from AnieeG March 14, 2024 14:56
@chainchad chainchad enabled auto-merge (squash) March 14, 2024 15:13
@chainchad chainchad merged commit c8d2749 into ccip-develop Mar 14, 2024
69 of 70 checks passed
@chainchad chainchad deleted the re-2277/check-for-ingress-dns-ccip branch March 14, 2024 15:31
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.

3 participants