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

Solr Broker User Permission Mark 2 #167

Merged
merged 1 commit into from
Nov 1, 2022

Conversation

nickumia-reisys
Copy link
Contributor

Related to

Fixes

  • Error: failure configuring LB attributes: AccessDenied: User: ssb-solr-broker is not authorized to perform: elasticloadbalancing:ModifyLoadBalancerAttributes on resource: loadbalancer/app/solr-lb/lb
  • Error: error listing tags for resource (namespace/ns-solr): AccessDeniedException: User: ssb-solr-broker is not authorized to perform: servicediscovery:ListTagsForResource on resource: servicediscovery:*/*

@nickumia-reisys nickumia-reisys temporarily deployed to staging November 1, 2022 14:44 Inactive
@nickumia-reisys nickumia-reisys temporarily deployed to production November 1, 2022 14:44 Inactive
@nickumia-reisys nickumia-reisys requested a review from a team November 1, 2022 14:44
@github-actions
Copy link

github-actions bot commented Nov 1, 2022

Terraform plan for production

Plan: 0 to add, 1 to change, 0 to destroy.
Terraform used the selected providers to generate the following execution
plan. Resource actions are indicated with the following symbols:
  ~ update in-place

Terraform will perform the following actions:

  # module.solr_brokerpak_policy.aws_iam_policy.policy will be updated in-place
  ~ resource "aws_iam_policy" "policy" {
        id          = "arn:aws:iam::821341638715:policy/solr_brokerpak_policy"
        name        = "solr_brokerpak_policy"
      ~ policy      = jsonencode(
          ~ {
              ~ Statement = [
                  ~ {
                      ~ Action   = [
                          + "elasticloadbalancing:ModifyLoadBalancerAttributes",
                            "iam:CreateUser",
                            # (54 unchanged elements hidden)
                            "logs:PutRetentionPolicy",
                          + "servicediscovery:ListTagsForResource",
                        ]
                        # (2 unchanged elements hidden)
                    },
                ]
                # (1 unchanged element hidden)
            }
        )
        tags        = {}
        # (5 unchanged attributes hidden)
    }

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

✅ Plan applied in apply #114

@github-actions
Copy link

github-actions bot commented Nov 1, 2022

Terraform plan for staging

Plan: 0 to add, 1 to change, 0 to destroy.
Terraform used the selected providers to generate the following execution
plan. Resource actions are indicated with the following symbols:
  ~ update in-place

Terraform will perform the following actions:

  # module.solr_brokerpak_policy.aws_iam_policy.policy will be updated in-place
  ~ resource "aws_iam_policy" "policy" {
        id          = "arn:aws:iam::054110281448:policy/solr_brokerpak_policy"
        name        = "solr_brokerpak_policy"
      ~ policy      = jsonencode(
          ~ {
              ~ Statement = [
                  ~ {
                      ~ Action   = [
                          + "elasticloadbalancing:ModifyLoadBalancerAttributes",
                            "iam:CreateUser",
                            # (54 unchanged elements hidden)
                            "logs:PutRetentionPolicy",
                          + "servicediscovery:ListTagsForResource",
                        ]
                        # (2 unchanged elements hidden)
                    },
                ]
                # (1 unchanged element hidden)
            }
        )
        tags        = {}
        # (5 unchanged attributes hidden)
    }

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

✅ Plan applied in apply #114

@nickumia-reisys nickumia-reisys merged commit a54d1b6 into main Nov 1, 2022
@nickumia-reisys nickumia-reisys deleted the add-permissions-to-solr-user-2 branch November 1, 2022 14:47
@nickumia-reisys nickumia-reisys changed the title More missing permissions for Solr Broker User Solr Broker User Permission Mark 2 Nov 1, 2022
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.

2 participants