Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Repo info
Activity
  • Oct 15 19:26
    miq-bot commented #757
  • Oct 15 19:21
    miq-bot labeled #757
  • Oct 15 19:20
    jrafanie closed #587
  • Oct 15 19:20
    jrafanie commented #587
  • Oct 15 19:17
    jrafanie review_requested #757
  • Oct 15 19:17
    jrafanie review_requested #757
  • Oct 15 19:17
    jrafanie labeled #757
  • Oct 15 19:17
    jrafanie opened #757
  • Oct 12 18:19
    Fryguy assigned #756
  • Oct 12 18:19
    Fryguy labeled #756
  • Oct 12 18:19

    Fryguy on master

    Remove unused supports methods … Merge pull request #756 from ag… (compare)

  • Oct 12 18:19
    Fryguy closed #756
  • Oct 12 17:51
    miq-bot commented #756
  • Oct 12 17:44
    agrare review_requested #756
  • Oct 12 17:44
    agrare opened #756
  • Oct 12 15:04
    chessbyte closed #755
  • Oct 11 16:56
    miq-bot commented #755
  • Oct 11 16:48
    chessbyte review_requested #755
  • Oct 11 16:48
    chessbyte review_requested #755
  • Oct 11 16:48
    chessbyte assigned #755
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
Thomas Buchinger
@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:
Thomas Buchinger
@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

Thomas Buchinger
@ThomasBuchinger
@Fryguy No Problem
@agrare I don't have the miq-Utilities domain in my environment, so I couldn't test which object it uses internally, it just looked similar, so I thoughtit might use the old object
if there are any method that you're missing (e.g. I don't see resize disk there, just add and remove) we should add them to the service models
Thomas Buchinger
@ThomasBuchinger
As I said, this was ages ago (even before I worked on our CloudForms). I guess there was no support for #add_disk back then? And we never bothered to fix it when there was support. We only need add_disk so we should be fine :)
Adam Grare
@agrare
actually there is no resize_disk in MIQ core, that does a direct reconfig :scream:
so yeah, we should add methods for that haha
Thomas Buchinger
@ThomasBuchinger
Always happy to add more stuff to your todo-list, sorry for that :sunglasses:
Adam Grare
@agrare
can you open an enhancement issue on the vmware repo? otherwise I'll forget it haha
and link to the miq-utilities method so we can update that to use the new one
thanks!