Skip to content

convert protocal names to be found as labels#3747

Merged
DaanHoogland merged 2 commits intoapache:4.13from
shapeblue:protocolsInUI
Dec 20, 2019
Merged

convert protocal names to be found as labels#3747
DaanHoogland merged 2 commits intoapache:4.13from
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?

@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-430

@yadvr yadvr added this to the 4.13.1.0 milestone Dec 7, 2019
Comment thread ui/scripts/network.js Outdated
@borisstoyanov
Copy link
Copy Markdown
Contributor

@blueorangutan package

@blueorangutan
Copy link
Copy Markdown

@borisstoyanov 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-471

Copy link
Copy Markdown
Contributor

@borisstoyanov borisstoyanov left a comment

Choose a reason for hiding this comment

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

LGTM, checked on mac with FF, Chrom and Safari.
Screenshot 2019-12-16 at 12 55 35

@DaanHoogland DaanHoogland merged commit 90ce1d8 into apache:4.13 Dec 20, 2019
@DaanHoogland DaanHoogland deleted the protocolsInUI branch December 20, 2019 13:11
DaanHoogland added a commit that referenced this pull request Dec 20, 2019
* create template from snapshot regression (partly reverted) (#3767)

* Once again allow a VM to be on multiple networks from VPCs (#3754)

to once again allow a VM to be on multiple networks from VPCs

* convert protocal names to be found as labels (#3747)

* convert protocal names to be found as labels

* format

* filter hosts to query on zone wide storage (#3733)

* config: add isdynamic flag in configuration response (#3729)

Co-authored-by: Wei Zhou <ustcweizhou@gmail.com>
DaanHoogland added a commit that referenced this pull request Dec 23, 2019
* 4.13:
  Added zone check for attach iso (#3755)
  config: add isdynamic flag in configuration response (#3729)
  filter hosts to query on zone wide storage (#3733)
  convert protocal names to be found as labels (#3747)
  Once again allow a VM to be on multiple networks from VPCs (#3754)
  create template from snapshot regression (partly reverted) (#3767)
ustcweizhou pushed a commit to ustcweizhou/cloudstack that referenced this pull request Feb 28, 2020
* convert protocal names to be found as labels

* format
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.

5 participants