provectus/kafka-ui

FE: KC: Removing KC cluster action is not disabled w/ r/o mode

Open

#3342 opened on Feb 13, 2023

View on GitHub
 (8 comments) (0 reactions) (1 assignee)Java (7,799 stars) (977 forks)batch import
area/uxgood first issuescope/frontendseverity/lowstatus/acceptedtype/bug

Description

Describe the bug (Actual behavior) I enable read only mode for Kafka Cluster. Under this mode I'm able to remove Kafka Connect connector.

Expected behavior I should't be able to remove Kafka Connect connector in read-only mode.

Set up

How do you run the app? Please provide as much info as possible:

  1. App version: v0.5.0 (027d9b4)
  2. Helm chart version, if you use one: kafka-ui-0.5.2
  3. Any IAAC configs
yamlApplicationConfig:
  kafka:
    clusters:
      - name: bds
        bootstrapServers: "kafka:9092"
        schemaRegistry: "http://kafka-cp-schema-registry.default.svc:8081"
        readOnly: true
        kafkaConnect:
        - name: connect
          address: "http://kafka-cp-kafka-connect.default.svc:8083"

Steps to Reproduce

  1. Go to Kafka Connect connectors.
  2. Click on ... next to connector.
  3. Remove it

Screenshots

image

Contributor guide

Tech stack
javareact
Domain
frontendbackend
Issue type
bug
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
3
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
1-3 hours
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
blocked
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
clear
Prerequisites
ReactJavaKafka Connectread only mode
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
40
Research direction
Investigate the frontend component responsible for the Kafka Connect connector list and actions. Locate the conditional that disables actions in read only mode (likely based on a readOnly flag). Ensure the remove action is included in the disabled set. Check the backend to confirm the readOnly flag is properly passed to the frontend.