Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Repo info
Activity
  • Apr 01 22:43
    agrare assigned #567
  • Apr 01 22:42
    agrare closed #567
  • Apr 01 22:42

    agrare on master

    provision: use the proper term … Merge pull request #567 from kb… (compare)

  • Apr 01 22:15
    miq-bot commented #567
  • Apr 01 22:07
    kbrock opened #567
  • Apr 01 22:07
    kbrock review_requested #567
  • Apr 01 22:07
    kbrock review_requested #567
  • Apr 01 16:06
    simaishi labeled #565
  • Apr 01 16:06
    simaishi unlabeled #565
  • Apr 01 16:06
    simaishi commented #565
  • Apr 01 16:06

    simaishi on jansa

    Merge pull request #565 from ag… (compare)

  • Mar 31 21:40
    miq-bot commented #566
  • Mar 31 21:40
    miq-bot commented #566
  • Mar 31 21:32
    agrare labeled #566
  • Mar 31 21:32
    agrare review_requested #566
  • Mar 31 21:32
    agrare opened #566
  • Mar 31 17:50

    chessbyte on master

    Fix targeted refresh of VM with… Merge pull request #565 from ag… (compare)

  • Mar 31 17:50
    chessbyte closed #565
  • Mar 31 17:50
    chessbyte assigned #565
  • Mar 31 17:50
    chessbyte unassigned #565
Adam Grare
@agrare
yeah that's the next step
was dependent on that one
wow gitter's being weird for me
Daniel Berger
@djberg96
along those lines, ManageIQ/manageiq-providers-vmware#417 needs a rebase it looks like
Adam Grare
@agrare
I need to actually write graph refresh for that one though
Adam Grare
@agrare
hey @djberg96 are there any ems_operations queue_name PRs to go?
looks like everything on that list is crossed off :)
Daniel Berger
@djberg96
miq_action.rb was the last one that I had
whether or not that's the last one needed, well, i think i'll have to defer to you
(just updated it, they're all crossed off now)
Adam Grare
@agrare
perfect thanks!
Adam Grare
@agrare
Thanks, yeah I have a branch to completely remove the broker
passes cross-repo tests
Daniel Berger
@djberg96
I take it you haven't pushed it
Adam Grare
@agrare
haven't pushed a PR yet
trying to break it up into smaller chunks
but my branch is pushed to my fork
Daniel Berger
@djberg96
ok, thanks
Daniel Berger
@djberg96
gubbe505
@gubbe505
Hello, the company that I work for is using ManageIQ with a VMware vSphere environment. We are exploring if we are going to use VMware NSX-T as SDN for bare-metal, hypervisor and VM alike. Are there any plans for a NSX-T network provider?
@agrare ^?
Adam Grare
@agrare
Yes, this was implemented by ManageIQ/manageiq-providers-vmware#449
it doesn't add an NSX network manager, rather it allows you to select the OpaqueNetworks in the VIM API which represent NSX network
gubbe505
@gubbe505
We would like to have something like the manageiq-providers-nuage. Some basic CRUD functionality to manage the SDN from within ManageIQ for ours tenants.
Adam Grare
@agrare
yeah understood, if you want to actually manage the networks we'd need a network manager
I don't have an NSX environment here to use, but if you want to help I'd happily work with you to get something like that added
(Its always something I've wanted to get working but never had the time or resources to)
gubbe505
@gubbe505
So, no plans for the foreseeable future??
Adam Grare
@agrare
not without help ;)
I could get the scaffolding done without access to the environment, but once we get to the parser I'd need someone like you to run it against a live system
gubbe505
@gubbe505
Oké. Great.
I have had some time to look at the datamodel of both MIQ and NSX-T. We want ours tenants to manage NSX-T segments, subnets, groups, services and security policies.
The segment, subnet and group can be mapped to ManageIQ models, but there is no model for the services and security policy.
Adam Grare
@agrare
hm looks like they only have python and java SDKs for NSX, that's going to be a bit of an issue
ThomasBuchinger
@ThomasBuchinger

@agrare Hi, just read your Post on vimBroker removal. is there a 1:1 replacement for people using vimBroker from Automate?

I think there are a many people using code like this vim = $evm.vmdb(:vm).first.object_send('instance_eval', 'with_provider_object { | vimVm | return vimVm }') to get a vimBrocker object and add additional disks to the VM. I have seen this bit of code in various iterations on the manageiq forum and in the RedHatOfficial/miq-Utilities repo. I think you guys might want to make sure everybody knows about this change and what they need to do
https://github.com/RedHatOfficial/miq-Utilities/blob/ef24e21875bb5527335a8110a3493bb8407abf4e/Automate/RedHatConsulting_Utilities/Infrastructure/VM/Provisioning/StateMachines/Methods.class/__methods__/resize_disk.rb

Jason Frey
@Fryguy
.object_send('instance_eval' was never supported

If you need resize_disk functionality not present in upstream I would recommend getting a PR into upstream with this code [1] to add it as a more official method

[1] https://github.com/RedHatOfficial/miq-Utilities/blob/ef24e21875bb5527335a8110a3493bb8407abf4e/Automate/RedHatConsulting_Utilities/Infrastructure/VM/Provisioning/StateMachines/Methods.class/__methods__/resize_disk.rb#L58

then you'd just call a normal ".resize_disk" method on the vm object without having to hack around the object boundaries
otherwise you can use rbvmomi and or vmware_web_service gems directly
Jason Frey
@Fryguy
@agrare Might be useful to reply to your own post with these alternatives
Adam Grare
@agrare
:+1:
ThomasBuchinger
@ThomasBuchinger

I know it is not supported. Correct me if I am wrong, but doesn't the resize_disk method use the vimBroker?

I just wanted to alert you to the fact that I have seen this bit of code in a few AutomateDomains and and that you might want to communicate this change in some official way. We got this piece of code from pemcg ages ago, so I think this could be pretty widely used

Adam Grare
@agrare
@ThomasBuchinger resize_disk is a method on the MiqVimVm object, previoiusly it was run in the broker but now the exact same code is run by the OperationsWorker
if you use the normal Vm#add_disk Vm#resize_disk method it will work
Jason Frey
@Fryguy

I just wanted to alert you to the fact that I have seen this bit of code in a few AutomateDomains and and that you might want to communicate this change in some official way. We got this piece of code from pemcg ages ago, so I think this could be pretty widely used

thanks for the heads up on that