Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added CIFS Support and fixed comparison statement for minimum Count of hosts #249

Closed
wants to merge 0 commits into from

Conversation

edv-pi
Copy link
Contributor

@edv-pi edv-pi commented Jul 6, 2024

and added support for namespaces in pbs, and ability to specify knet_priority for cororsync via pve_cluster_prio0 and 1 (In my case i wanted to be able to use second link for sperate cororsync traffic with own switch but wanted still the failover to my public network if switch goes down

Copy link
Owner

@lae lae left a comment

Choose a reason for hiding this comment

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

Please rebase all of these commits into logical ones (i.e. no test/typo fix commits and no merge commits) and use descriptive commit messages for the initial line (one commit per feature/change, e.g. 081b98a is a good one even though it kind of runs a bit long). You can force push to this PR when done with the rebase.

@lae
Copy link
Owner

lae commented Jul 7, 2024

I went ahead and moved 1eb0256 into a separate PR (#250) because that's a critical bug. Since it's been merged, you should be able to leave it out when rebasing.

@lae
Copy link
Owner

lae commented Jul 10, 2024

Replaced by #251

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