Skip to content

add /api/v1/test-list/tor-targets to load balancer#379

Merged
aagbsn merged 1 commit intomainfrom
add_tor_targets_to_load_balancer
Mar 19, 2026
Merged

add /api/v1/test-list/tor-targets to load balancer#379
aagbsn merged 1 commit intomainfrom
add_tor_targets_to_load_balancer

Conversation

@aagbsn
Copy link
Contributor

@aagbsn aagbsn commented Mar 19, 2026

No description provided.

@github-actions
Copy link

Terraform Run Output 🤖

Format and Style 🖌success

Initialization ⚙️success

Validation 🤖success

Validation Output

$ terraform validate

Warning: Argument is deprecated

  with aws_s3_bucket.anoncred_manifests,
  on main.tf line 255, in resource "aws_s3_bucket" "anoncred_manifests":
 255: resource "aws_s3_bucket" "anoncred_manifests" {

Use the aws_s3_bucket_versioning resource instead

Warning: Available Write-only Attribute Alternative

  with module.ooni_monitoring.aws_ssm_parameter.ooni_monitoring_access_key,
  on ../../modules/ooni_monitoring/main.tf line 47, in resource "aws_ssm_parameter" "ooni_monitoring_access_key":
  47:   value = aws_iam_access_key.ooni_monitoring.id

The attribute value has a write-only alternative value_wo available. Use the
write-only alternative of the attribute when possible.

(and one more similar warning elsewhere)
Success! The configuration is valid, but there were some validation warnings
as shown above.

Plan 📖success

  • undefined
Show Plan

$ terraform plan
Acquiring state lock. This may take a few moments...
Pusher @aagbsn
Action pull_request
Environment dev
Workflow .github/workflows/check_terraform.yml
Last updated Thu, 19 Mar 2026 12:54:49 GMT

Copy link
Contributor

@LDiazN LDiazN left a comment

Choose a reason for hiding this comment

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

LGTM 🚀

@aagbsn aagbsn merged commit eb7f35f into main Mar 19, 2026
2 checks passed
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.

2 participants