Skip to content

Conversation

rwinieski
Copy link
Collaborator

No description provided.

Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR fixes a bug related to FluentBit logs volume mounting by correcting the mount path for the "datadir" volume mount to specifically point to the Logs subdirectory.

  • Updated the FluentBit volume mount path from /var/opt/MarkLogic to /var/opt/MarkLogic/Logs

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Copy link
Collaborator

@pengzhouml pengzhouml left a comment

Choose a reason for hiding this comment

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

@rwinieski , I checked with Helm Chart implementation, it is also using /var/opt/MarkLogic. The only difference is that in Helm chart we also add the volume.
Below is the code that currently in our Helm Chart:
volumeMounts:
- name: datadir
mountPath: /var/opt/MarkLogic
{{- if .Values.additionalVolumeMounts }}
{{- toYaml .Values.additionalVolumeMounts | nindent 12 }}
{{- end }}

@rwinieski
Copy link
Collaborator Author

@rwinieski , I checked with Helm Chart implementation, it is also using /var/opt/MarkLogic. The only difference is that in Helm chart we also add the volume. Below is the code that currently in our Helm Chart: volumeMounts: - name: datadir mountPath: /var/opt/MarkLogic {{- if .Values.additionalVolumeMounts }} {{- toYaml .Values.additionalVolumeMounts | nindent 12 }} {{- end }}

That what is done here as well:

	VolumeMountsFluentBit = append(VolumeMountsFluentBit,
		corev1.VolumeMount{
			Name:      "datadir",
			MountPath: "/var/opt/MarkLogic/Logs",
		},

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