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

Add ipv4 ipv6 selection to Subnet view #12650

Merged

Conversation

andyvesel
Copy link
Contributor

@andyvesel andyvesel commented Nov 15, 2016

https://bugzilla.redhat.com/show_bug.cgi?id=1394286

  1. Added a ipv4/ipv6 selection field to Subnet create page
    ip_version

  2. Locked ipv4/ipv6 and CIDR fields on Subnet edit page
    lock

@chessbyte
Copy link
Member

Screenshots please

@dclarizio
Copy link

@andyvesel please rebase and repush, I think the expression tests were fixed in another PR. Also, agree with @chessbyte, needs screenshots. Thx, Dan

@andyvesel andyvesel force-pushed the add_ipv4_ipv6_selection_to_subnet_view branch from 22cf5cc to c56126c Compare November 18, 2016 10:15
@dclarizio
Copy link

@andyvesel should the IP version be a drop down choice of 4 and 6? Also, should there be some code to handle the new value coming in?
@Ladas can you please review?

@Ladas
Copy link
Contributor

Ladas commented Nov 21, 2016

@andyvesel @dclarizio yes, it should be a dropdown, I don't expect anything else than IPv4 and IPv6 can be really there? ( at least for a next century :-) )

Backend was already merged? Usually we mark the backed PR as a dependency and a UI PR is a WIP until backend is merged

@andyvesel
Copy link
Contributor Author

andyvesel commented Nov 22, 2016

@Ladas, is this the right backend PR?
#12561

@Ladas
Copy link
Contributor

Ladas commented Nov 22, 2016

@andyvesel doesn't seems to be, those are just logs. With you PR, does it actually store the right IP version to OpenStack?

@andyvesel andyvesel changed the title Add ipv4 ipv6 selection to Subnet view [WIP] Add ipv4 ipv6 selection to Subnet view Nov 23, 2016
@chessbyte chessbyte added the wip label Nov 23, 2016
@andyvesel andyvesel force-pushed the add_ipv4_ipv6_selection_to_subnet_view branch from c56126c to 7602272 Compare November 23, 2016 15:35
@miq-bot
Copy link
Member

miq-bot commented Nov 23, 2016

Checked commits andyvesel/manageiq@bd4d1e6~...7602272 with ruby 2.2.5, rubocop 0.37.2, and haml-lint 0.16.1
0 files checked, 0 offenses detected
Everything looks good. 👍

@andyvesel andyvesel changed the title [WIP] Add ipv4 ipv6 selection to Subnet view Add ipv4 ipv6 selection to Subnet view Nov 29, 2016
@andyvesel
Copy link
Contributor Author

@Ladas can you review please?

@Ladas
Copy link
Contributor

Ladas commented Nov 30, 2016

looks good 👍

@chessbyte chessbyte removed the wip label Dec 1, 2016
@dclarizio
Copy link

@andyvesel if this was changed to a drop down, can you replace the screen shot above? Thx, Dan

@andyvesel
Copy link
Contributor Author

@dclarizio done

@dclarizio dclarizio merged commit 464fe05 into ManageIQ:master Dec 6, 2016
@dclarizio dclarizio added this to the Sprint 51 Ending Jan 2, 2017 milestone Dec 6, 2016
@andyvesel andyvesel deleted the add_ipv4_ipv6_selection_to_subnet_view branch December 7, 2016 08:43
@tzumainn
Copy link
Contributor

@miq-bot add_label blocker
@miq-bot add_label bug

simaishi pushed a commit that referenced this pull request Jan 9, 2017
…ubnet_view

Add ipv4 ipv6 selection to Subnet view
(cherry picked from commit 464fe05)

https://bugzilla.redhat.com/show_bug.cgi?id=1402162
@simaishi
Copy link
Contributor

simaishi commented Jan 9, 2017

Euwe backport details:

$ git log -1
commit b8909f2b50f60f00b49180d570a8d2a079e340d6
Author: Dan Clarizio <dclarizi@redhat.com>
Date:   Tue Dec 6 14:21:12 2016 -0500

    Merge pull request #12650 from andyvesel/add_ipv4_ipv6_selection_to_subnet_view
    
    Add ipv4 ipv6 selection to Subnet view
    (cherry picked from commit 464fe051be7885b2bc4f8cd6a2ed4044f6b4721f)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1402162

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.

7 participants