Skip to content

Add notice about https://github.com/stackabletech/hdfs-operator/issues/274 containing a breaking change to the release notes#348

Merged
fhennig merged 1 commit into
mainfrom
fix/hdfs_breaking_change
Jan 26, 2023
Merged

Add notice about https://github.com/stackabletech/hdfs-operator/issues/274 containing a breaking change to the release notes#348
fhennig merged 1 commit into
mainfrom
fix/hdfs_breaking_change

Conversation

@soenkeliebau
Copy link
Copy Markdown
Member

No description provided.

Copy link
Copy Markdown
Member

@adwk67 adwk67 left a comment

Choose a reason for hiding this comment

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

lgtm

@fhennig fhennig merged commit 407bd8a into main Jan 26, 2023
@fhennig fhennig deleted the fix/hdfs_breaking_change branch January 26, 2023 13:03
Copy link
Copy Markdown
Member

@maltesander maltesander left a comment

Choose a reason for hiding this comment

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

Should we mention to "stop" the datanodes by setting replicas to 0?


[source,bash]
----
export PVNAME=$(kubectl get pvc hdfs-datanode-default-0 -o yaml | yq '.spec.volumeName')
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I would mention the yq version. For me on 4.17 i have to do (this solution requires 4.18+):
export PVNAME=$(kubectl get pvc hdfs-datanode-default-0 -o yaml | yq e '.spec.volumeName' -)
and maybe add the namespace like:
export PVNAME=$(kubectl get -n <namespace> pvc hdfs-datanode-default-0 -o yaml | yq '.spec.volumeName')

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Well, too late

fhennig pushed a commit that referenced this pull request Jan 26, 2023
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.

4 participants