Skip to content

convert protocol names to be found as labels#3725

Closed
DaanHoogland wants to merge 1 commit into
apache:masterfrom
shapeblue:protocolsInUI
Closed

convert protocol names to be found as labels#3725
DaanHoogland wants to merge 1 commit into
apache:masterfrom
shapeblue:protocolsInUI

Conversation

@DaanHoogland
Copy link
Copy Markdown
Contributor

Description

replace significant chars from string to find protocol labels

Fixes: #3252

Types of changes

  • Breaking change (fix or feature that would cause existing functionality to change)
  • New feature (non-breaking change which adds functionality)
  • Bug fix (non-breaking change which fixes an issue)
  • Enhancement (improves an existing feature and functionality)
  • Cleanup (Code refactoring and cleanup, that may add test cases)

Screenshots (if appropriate):

image

How Has This Been Tested?

@GabrielBrascher GabrielBrascher added this to the 4.14.0.0 milestone Nov 28, 2019
@GabrielBrascher GabrielBrascher self-requested a review November 28, 2019 21:20
@DaanHoogland DaanHoogland changed the title convert protocal names to be found as labels convert protocol names to be found as labels Nov 29, 2019
@DaanHoogland
Copy link
Copy Markdown
Contributor Author

@blueorangutan package

@blueorangutan
Copy link
Copy Markdown

@DaanHoogland a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

@blueorangutan
Copy link
Copy Markdown

Packaging result: ✖centos6 ✔centos7 ✔debian. JID-407

@DaanHoogland DaanHoogland marked this pull request as ready for review December 5, 2019 15:26
@DaanHoogland
Copy link
Copy Markdown
Contributor Author

reviewers please consider if this is for the right milestone.

@DaanHoogland DaanHoogland marked this pull request as ready for review December 5, 2019 15:29
@DaanHoogland DaanHoogland changed the base branch from master to 4.13 December 6, 2019 12:49
@DaanHoogland DaanHoogland changed the base branch from 4.13 to master December 6, 2019 12:50
@DaanHoogland
Copy link
Copy Markdown
Contributor Author

DaanHoogland commented Dec 6, 2019

needs to be on 4.13??

@GabrielBrascher
Copy link
Copy Markdown
Member

@DaanHoogland If we consider it as a bug then it makes sense to have it fixed on 4.13.1.0. However, if it is considered an enhancement then I don't see why not keeping the 4.14 milestone.

Looking at #3252 I would consider it as a bug on the UI; +1 on setting it to branch 4.13 and milestone 4.13.1.0.

@DaanHoogland
Copy link
Copy Markdown
Contributor Author

rebasing on 4.13 in #3747

@DaanHoogland DaanHoogland removed their assignment Jun 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Load balancer protocols bug

4 participants