feedback-to-code/mozzarella huggingface dataset:
Repository: provectus/kafka-ui
Issue:  Unclear parameters for Consumers` Search. Inconsistency of Search line on Consumers page.
Labels: type/enhancement
scope/frontend
status/accepted
status/confirmed

### Is your proposal related to a problem?

<!--
  Provide a clear and concise description of what the problem is.
  For example, "I'm always frustrated when..."
-->

(Write your answer here.)
It`s confusing now what parameter the search is currently using because it`s only one word 'Search' into the search line on Consumers page. Moreover, the Search line is inconsistent here - as on other pages there is a clear hint into the search line like Search by [parameters]

### Describe the solution you'd like

<!--
  Provide a clear and concise description of what you want to happen.
-->

(Describe your proposed solution here.)
The clear hint in the Search line for instance Search by Consumer group ID

### Describe alternatives you've considered

<!--
  Let us know about other solutions you've tried or researched.
-->

(Write your answer here.)

### Additional context

<!--
  Is there anything else you can add about the proposal?
  You might want to link to related issues here, if you haven't already.
-->

(Write your answer here.)


Issue comments:

Pull request review comments:

Patch that resolved the issue:
diff --git a/kafka-ui-react-app/src/components/ConsumerGroups/List/List.tsx b/kafka-ui-react-app/src/components/ConsumerGroups/List/List.tsx
index f5b006a1c6d..9cd37695850 100644
--- a/kafka-ui-react-app/src/components/ConsumerGroups/List/List.tsx
+++ b/kafka-ui-react-app/src/components/ConsumerGroups/List/List.tsx
@@ -22,7 +22,7 @@ const List: React.FC = () => {
       <PageHeading text="Consumers" />
       <ControlPanelWrapper hasInput>
         <Search
-          placeholder="Search"
+          placeholder="Search by Consumer Group ID"
           value={searchText}
           handleSearch={handleInputChange}
         />
diff --git a/kafka-ui-react-app/src/components/ConsumerGroups/List/__test__/List.spec.tsx b/kafka-ui-react-app/src/components/ConsumerGroups/List/__test__/List.spec.tsx
index 1bcb34daac9..0ac9c0ceedf 100644
--- a/kafka-ui-react-app/src/components/ConsumerGroups/List/__test__/List.spec.tsx
+++ b/kafka-ui-react-app/src/components/ConsumerGroups/List/__test__/List.spec.tsx
@@ -30,7 +30,10 @@ describe('List', () => {
 
     describe('when searched', () => {
       it('renders only searched consumers', () => {
-        userEvent.type(screen.getByPlaceholderText('Search'), 'groupId1');
+        userEvent.type(
+          screen.getByPlaceholderText('Search by Consumer Group ID'),
+          'groupId1'
+        );
         expect(screen.getByText('groupId1')).toBeInTheDocument();
         expect(screen.getByText('groupId2')).toBeInTheDocument();
       });


Test patch:
Repository: provectus/kafka-ui
Issue: Tag master docker image with a version number tag
Labels: type/enhancement
status/accepted
scope/infrastructure

The ":latest" tag does not have the latest.  It looks like the most recent image is only tagged as ":master".  It would be nice if it was tagged with a version number.  We like to specify a specific version so we know what we are installing.  The version from 3 months ago was tagged as 0.23.1.  Looking for something similar with the most recent image.



Issue comments:
Hello there swilliams-a3digital! 👋

Thank you and congratulations 🎉 for opening your very first issue in this project! 💖

In case you want to claim this issue, please comment down below! We will try to get back to you as soon as we can. 👀
Hi, thanks for reaching out. The thing is, `latest` points to latest *release* version.
We'll consider additionally labelling each commit with a dockerhub label as well.
Btw, there's a 0.3 release coming out soon (this week).

Pull request review comments:

Patch that resolved the issue:
diff --git a/.github/workflows/master.yaml b/.github/workflows/master.yaml
index fd94ae63c27..9d610eab3cb 100644
--- a/.github/workflows/master.yaml
+++ b/.github/workflows/master.yaml
@@ -63,7 +63,9 @@ jobs:
           context: kafka-ui-api
           platforms: linux/amd64,linux/arm64
           push: true
-          tags: provectuslabs/kafka-ui:master
+          tags: |
+            provectuslabs/kafka-ui:${{ steps.build.outputs.version }}
+            provectuslabs/kafka-ui:master
           build-args: |
             JAR_FILE=kafka-ui-api-${{ steps.build.outputs.version }}.jar
           cache-from: type=local,src=/tmp/.buildx-cache


Test patch:
Repository: provectus/kafka-ui
Issue: Tag master docker image with a version number tag
Labels: type/enhancement
status/accepted
scope/infrastructure

The ":latest" tag does not have the latest.  It looks like the most recent image is only tagged as ":master".  It would be nice if it was tagged with a version number.  We like to specify a specific version so we know what we are installing.  The version from 3 months ago was tagged as 0.23.1.  Looking for something similar with the most recent image.



Issue comments:
Hello there swilliams-a3digital! 👋

Thank you and congratulations 🎉 for opening your very first issue in this project! 💖

In case you want to claim this issue, please comment down below! We will try to get back to you as soon as we can. 👀
Hi, thanks for reaching out. The thing is, `latest` points to latest *release* version.
We'll consider additionally labelling each commit with a dockerhub label as well.
Btw, there's a 0.3 release coming out soon (this week).

Pull request review comments:

Patch that resolved the issue:
diff --git a/.github/workflows/release.yaml b/.github/workflows/release.yaml
index a88c7d05d81..12c11a771d2 100644
--- a/.github/workflows/release.yaml
+++ b/.github/workflows/release.yaml
@@ -109,6 +109,7 @@ jobs:
         run: |
           export version=${{needs.release.outputs.version}}
           sed -i "s/version:.*/version: ${version}/" charts/kafka-ui/Chart.yaml
+          sed -i "s/appVersion:.*/appVersion: ${version}/" charts/kafka-ui/Chart.yaml
 
       - name: add chart
         run: |
diff --git a/charts/kafka-ui/values.yaml b/charts/kafka-ui/values.yaml
index 3fc0754aff5..43cbf869f6b 100644
--- a/charts/kafka-ui/values.yaml
+++ b/charts/kafka-ui/values.yaml
@@ -5,7 +5,7 @@ image:
   repository: provectuslabs/kafka-ui
   pullPolicy: IfNotPresent
   # Overrides the image tag whose default is the chart appVersion.
-  tag: "latest"
+  tag: ""
 
 imagePullSecrets: []
 nameOverride: ""


Test patch:
Repository: provectus/kafka-ui
Issue: remove branch kafka-ui application from argocd when labeled pr is closed
Labels: type/enhancement
scope/infrastructure

RemoveCustomDeployment workflow is being triggered only when PR becomes unlabeled. However, when a pr is closed and is still labeled at the same time, an environment stays in ARGOCD.

The goal is to modify branch-remove.yml in order to be triggered on PR closure as well.

Issue comments:

Pull request review comments:

Patch that resolved the issue:
diff --git a/.github/workflows/branch-remove.yml b/.github/workflows/branch-remove.yml
index ad6e63aa71e..3d08e5682dd 100644
--- a/.github/workflows/branch-remove.yml
+++ b/.github/workflows/branch-remove.yml
@@ -2,7 +2,7 @@ name: RemoveCustomDeployment
 on:
   workflow_dispatch:
   pull_request:
-    types: ['unlabeled']
+    types: ['unlabeled', 'closed']
 jobs:
   remove:
     if: ${{ github.event.label.name == 'status/feature_testing' || github.event.label.name == 'status/feature_testing_public' }}
@@ -37,4 +37,4 @@ jobs:
         with:
           issue-number: ${{ github.event.pull_request.number }}
           body: |
-            Custom deployment removed
\ No newline at end of file
+            Custom deployment removed


Test patch:
Repository: provectus/kafka-ui
Issue: Make helm Ingress compatible with k8s v1.22+
Labels: type/enhancement
scope/infrastructure
scope/k8s

### Is your proposal related to a problem?
Can we change the ingress to make compatiable with k8s v1.22+
<!--
  Provide a clear and concise description of what the problem is.
  networking.k8s.io/v1beta1 Ingress is deprecated in v1.19+, unavailable in v1.22+; use networking.k8s.io/v1 Ingress
-->

(Write your answer here.)

### Describe the solution you'd like
Can we change the ingress to make compatiable with k8s v1.22+
<!--
  Provide a clear and concise description of what you want to happen.
-->

(Describe your proposed solution here.)

### Describe alternatives you've considered

<!--
  Let us know about other solutions you've tried or researched.
-->

(Write your answer here.)

### Additional context

<!--
  Is there anything else you can add about the proposal?
  You might want to link to related issues here, if you haven't already.
-->

(Write your answer here.)


Issue comments:
Hello there bahaeddinoz! 👋

Thank you and congratulations 🎉 for opening your very first issue in this project! 💖

In case you want to claim this issue, please comment down below! We will try to get back to you as soon as we can. 👀
Hey, thanks for reaching out. We'll take a look.
Thanks @Haarolean for the so quick updates 🙏 Just one question why directly didn't we use the "networking.k8s.io/v1 " and instead specified if k8s version is highter then 1.22 ? k8s supports it since v1.10. Can be good to use it directly or is there any other reason ?
https://kubernetes.io/docs/reference/using-api/deprecation-guide/#apiservice-v122
@bahaeddinoz first of all, your documentation says it's supported since 1.19, not 1.10 (`Migrate manifests and API clients to use the networking.k8s.io/v1 API version, available since v1.19.`). The main purpose is to save back compatibility.

Pull request review comments:

Patch that resolved the issue:
diff --git a/charts/kafka-ui/templates/ingress.yaml b/charts/kafka-ui/templates/ingress.yaml
index d6e6450dc57..e602bb7471d 100644
--- a/charts/kafka-ui/templates/ingress.yaml
+++ b/charts/kafka-ui/templates/ingress.yaml
@@ -3,6 +3,8 @@
 {{- $svcPort := .Values.service.port -}}
 {{- if semverCompare ">=1.14-0" .Capabilities.KubeVersion.GitVersion -}}
 apiVersion: networking.k8s.io/v1beta1
+{{- else if semverCompare ">=1.22-0" .Capabilities.KubeVersion.GitVersion -}}
+apiVersion: networking.k8s.io/v1
 {{- else -}}
 apiVersion: extensions/v1beta1
 {{- end }}


Test patch:
Repository: provectus/kafka-ui
Issue: Empty screen if try to Edit schema of proto type
Labels: type/bug
status/duplicate
scope/frontend

**Describe the bug**
(A clear and concise description of what the bug is.)

There is an empty screen when you try to edit the schema of proto type.

**Set up**
(How do you run the app? 
Which version of the app are you running? Provide either docker image version or check commit hash at the top left corner. We won't be able to help you without this information.)

http://dependabot-npm-and-yarn-kafka-ui.internal.kafka-ui.provectus.io/
**Steps to Reproduce**
Steps to reproduce the behavior:

1. Navigate to Schema Registry 
2. Click the schema with a name containing 'proto'
3. Press the button Edit Schema
4. Observe the empty screen with the right URL (see attachment)

**Expected behavior**
(A clear and concise description of what you expected to happen)
Edit screen for the schema.
**Screenshots**
(If applicable, add screenshots to help explain your problem)

![image](https://user-images.githubusercontent.com/92585878/149402931-43117037-cb89-4716-82c2-ffef381b4f60.png)

**Additional context**
(Add any other context about the problem here)
Works correctly for other schema types.

Issue comments:
No failed backend requests
A duplicate of #1188

Pull request review comments:
I think it better to use `SchemaType.PROTOBUF` from `import { SchemaType } from 'generated-sources';
` instead of hardcoded value

Patch that resolved the issue:
diff --git a/kafka-ui-react-app/src/components/Schemas/Edit/Edit.tsx b/kafka-ui-react-app/src/components/Schemas/Edit/Edit.tsx
index 63a48580dca..27a109f0238 100644
--- a/kafka-ui-react-app/src/components/Schemas/Edit/Edit.tsx
+++ b/kafka-ui-react-app/src/components/Schemas/Edit/Edit.tsx
@@ -37,10 +37,11 @@ const Edit: React.FC = () => {
 
   const schema = useAppSelector((state) => selectSchemaById(state, subject));
 
-  const formatedSchema = React.useMemo(
-    () => JSON.stringify(JSON.parse(schema?.schema || '{}'), null, '\t'),
-    [schema]
-  );
+  const formatedSchema = React.useMemo(() => {
+    return schema?.schemaType === SchemaType.PROTOBUF
+      ? schema?.schema
+      : JSON.stringify(JSON.parse(schema?.schema || '{}'), null, '\t');
+  }, [schema]);
 
   const onSubmit = React.useCallback(async (props: NewSchemaSubjectRaw) => {
     if (!schema) return;


Test patch:
Repository: provectus/kafka-ui
Issue: Missing Ingress pathType with k8s >=1.22
Labels: type/bug
status/accepted
status/confirmed
scope/k8s

**Describe the bug**
<!--(A clear and concise description of what the bug is.)-->
When installing the helm chart on K8s version >= 1.22 with ingress enabled, the path type is missing and gives the following message:
`Error: INSTALLATION FAILED: unable to build kubernetes objects from release manifest: error validating "": error validating data: ValidationError(Ingress.spec.rules[0].http.paths[0]): missing required field "pathType" in io.k8s.api.networking.v1.HTTPIngressPath`

**Set up**
<!--
(How do you run the app? 
Which version of the app are you running? Provide either docker image version or check commit hash at the top left corner. We won't be able to help you without this information.)
-->
** Needs to be done on a k8s cluster version 1.22 or greater.

Enable the ingress and specify a path in values.yaml:
```
kafka-ui:
  ingress:
    enabled: true
    path: /
```

**Steps to Reproduce**
Steps to reproduce the behavior:

1. `helm install chart-name .`

**Expected behavior**
<!--
(A clear and concise description of what you expected to happen)
-->
Chart is successfully installed is correctly formatted ingress.



Issue comments:
Hello there PhilDulac! 👋

Thank you and congratulations 🎉 for opening your very first issue in this project! 💖

In case you want to claim this issue, please comment down below! We will try to get back to you as soon as we can. 👀
@PhilDulac hey, thanks for reaching out. This has been fixed in `master`-labeled image, we'll also get a minor release with this change to update the charts on artifacthub.

Pull request review comments:

Patch that resolved the issue:
diff --git a/charts/kafka-ui/templates/ingress.yaml b/charts/kafka-ui/templates/ingress.yaml
index 641cfd9b67e..7c1c046a0eb 100644
--- a/charts/kafka-ui/templates/ingress.yaml
+++ b/charts/kafka-ui/templates/ingress.yaml
@@ -33,7 +33,7 @@ spec:
 {{- if semverCompare ">=1.22-0" .Capabilities.KubeVersion.GitVersion -}}
           {{- range .Values.ingress.precedingPaths }}
           - path: {{ .path }}
-            pathType: ImplementationSpecific
+            pathType: Prefix
             backend:
               service:
                 name: {{ .serviceName }}
@@ -45,12 +45,13 @@ spec:
                 name: {{ $fullName }}
                 port:
                   number: {{ $svcPort }}
+            pathType: Prefix
 {{- if .Values.ingress.path }}
             path: {{ .Values.ingress.path }}
 {{- end }}
           {{- range .Values.ingress.succeedingPaths }}
           - path: {{ .path }}
-            pathType: ImplementationSpecific
+            pathType: Prefix
             backend:
               service:
                 name: {{ .serviceName }}


Test patch:
Repository: provectus/kafka-ui
Issue: Add built jar file to a release
Labels: type/enhancement
scope/infrastructure

Need to attach built jar file to a published release.

Issue comments:

Pull request review comments:

Patch that resolved the issue:
diff --git a/.github/workflows/release.yaml b/.github/workflows/release.yaml
index 883c2937b8f..f0c3b51cc57 100644
--- a/.github/workflows/release.yaml
+++ b/.github/workflows/release.yaml
@@ -37,6 +37,13 @@ jobs:
           export VERSION=$(mvn -q -Dexec.executable=echo -Dexec.args='${project.version}' -DbuildVersion=${{ github.event.release.tag_name }} --non-recursive exec:exec)
           echo ::set-output name=version::${VERSION}
 
+      - name: Upload files to a GitHub release
+        uses: svenstaro/upload-release-action@2.2.1
+        with:
+          repo_token: ${{ secrets.GITHUB_TOKEN }}
+          file: kafka-ui-api/target/kafka-ui-api-${{ steps.build.outputs.version }}.jar
+          tag: ${{ github.event.release.tag_name }}
+
       - name: Archive JAR
         uses: actions/upload-artifact@v2
         with:


Test patch:
Repository: provectus/kafka-ui
Issue: Fix release and tag naming
Labels: type/bug
status/accepted
status/confirmed
scope/infrastructure

Need to name release and tag as it was previously.
```
Release name: 0.3.2
Tag name: v0.3.2
```

Issue comments:

Pull request review comments:

Patch that resolved the issue:
diff --git a/.github/release_drafter.yaml b/.github/release_drafter.yaml
index 21ad76787bc..952d2b5d905 100644
--- a/.github/release_drafter.yaml
+++ b/.github/release_drafter.yaml
@@ -1,5 +1,5 @@
-name-template: 'v$RESOLVED_VERSION'
-tag-template: '$RESOLVED_VERSION'
+name-template: '$RESOLVED_VERSION'
+tag-template: 'v$RESOLVED_VERSION'
 template: |
   ## Changes
   $CHANGES


Test patch:
Repository: provectus/kafka-ui
Issue: [Bug] Fix workflow for documentation
Labels: type/bug
status/accepted
status/confirmed
scope/infrastructure

Documentation is not triggered due to incorrect path filters. Need to fix.

Issue comments:

Pull request review comments:

Patch that resolved the issue:
diff --git a/.github/workflows/documentation.yaml b/.github/workflows/documentation.yaml
index 2a1171394dc..464af01fe80 100644
--- a/.github/workflows/documentation.yaml
+++ b/.github/workflows/documentation.yaml
@@ -7,8 +7,8 @@ on:
       - reopened
       - synchronize
     paths:
-      - 'documentation/*'
-      - './*.md'
+      - 'documentation/**'
+      - '**.md'
 jobs:
   build-and-test:
     runs-on: ubuntu-latest


Test patch:
Repository: provectus/kafka-ui
Issue: Trigger backend workflow if pom.xml in the repo root is changed
Labels: type/bug
status/accepted
status/confirmed
scope/infrastructure

pom.xml
mvnw?
mvnw.cmd?

Issue comments:

Pull request review comments:

Patch that resolved the issue:
diff --git a/.github/workflows/backend.yml b/.github/workflows/backend.yml
index 4cebf3557cb..73663a5392b 100644
--- a/.github/workflows/backend.yml
+++ b/.github/workflows/backend.yml
@@ -7,6 +7,7 @@ on:
     types: ["opened", "edited", "reopened", "synchronize"]
     paths:
       - "kafka-ui-api/**"
+      - "pom.xml"
 jobs:
   build-and-test:
     runs-on: ubuntu-latest
diff --git a/.github/workflows/e2e-checks.yaml b/.github/workflows/e2e-checks.yaml
index 832b07c85c8..00581514dfb 100644
--- a/.github/workflows/e2e-checks.yaml
+++ b/.github/workflows/e2e-checks.yaml
@@ -7,6 +7,7 @@ on:
       - "kafka-ui-contract/**"
       - "kafka-ui-react-app/**"
       - "kafka-ui-e2e-checks/**"
+      - "pom.xml"
 jobs:
   build-and-test:
     runs-on: ubuntu-latest


Test patch:
Repository: provectus/kafka-ui
Issue: Implement a workflow for periodic CVE scanning
Labels: scope/infrastructure

With a manual run so far.

#1656 
#1240 

Issue comments:

Pull request review comments:

Patch that resolved the issue:
diff --git a/.github/workflows/cve.yaml b/.github/workflows/cve.yaml
new file mode 100644
index 00000000000..8a8c2ac09ed
--- /dev/null
+++ b/.github/workflows/cve.yaml
@@ -0,0 +1,68 @@
+name: CVE checks docker master
+on:
+  workflow_dispatch:
+  schedule:
+    # * is a special character in YAML so you have to quote this string
+    - cron:  '0 8 15 * *'
+jobs:
+  build-and-test:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v2
+
+      - name: Cache local Maven repository
+        uses: actions/cache@v2
+        with:
+          path: ~/.m2/repository
+          key: ${{ runner.os }}-maven-${{ hashFiles('**/pom.xml') }}
+          restore-keys: |
+            ${{ runner.os }}-maven-
+
+      - name: Set up JDK 1.13
+        uses: actions/setup-java@v1
+        with:
+          java-version: 1.13
+
+      - name: Build project
+        id: build
+        run: |
+          mvn versions:set -DnewVersion=$GITHUB_SHA
+          mvn clean package -DskipTests
+          export VERSION=$(mvn -q -Dexec.executable=echo -Dexec.args='${project.version}' --non-recursive exec:exec)
+          echo "::set-output name=version::${VERSION}"
+
+      - name: Set up QEMU
+        uses: docker/setup-qemu-action@v1
+
+      - name: Set up Docker Buildx
+        uses: docker/setup-buildx-action@v1
+
+      - name: Cache Docker layers
+        uses: actions/cache@v2
+        with:
+          path: /tmp/.buildx-cache
+          key: ${{ runner.os }}-buildx-${{ github.sha }}
+          restore-keys: |
+            ${{ runner.os }}-buildx-
+
+      - name: Build docker image
+        uses: docker/build-push-action@v2
+        with:
+          builder: ${{ steps.buildx.outputs.name }}
+          context: kafka-ui-api
+          platforms: linux/amd64
+          push: false
+          load: true
+          tags: |
+            provectuslabs/kafka-ui:${{ steps.build.outputs.version }}
+          build-args: |
+            JAR_FILE=kafka-ui-api-${{ steps.build.outputs.version }}.jar
+          cache-from: type=local,src=/tmp/.buildx-cache
+          cache-to: type=local,dest=/tmp/.buildx-cache
+
+      - name: Run CVE checks
+        uses: aquasecurity/trivy-action@0.2.2
+        with:
+          image-ref: "provectuslabs/kafka-ui:${{ steps.build.outputs.version }}"
+          format: "table"
+          exit-code: "1"


Test patch: