[00:26] <hamalq> falcojr: we are trying to keep the default removing select_hostname from rhel will cause issues with default behaviour
[00:27] <falcojr> I wasn't suggesting changing default behavior
[00:28] <falcojr> I'll write up a diff
[00:33] <hamalq> the idea if we remove the select_hostname from rhel ( which prefer fqdn ) it will be back to distro select_hostname ( which prefer hostname)
[00:34] <hamalq> and freedbsd the same which will cause the behavior change
[00:34] <hamalq> falcojr: ^^
[00:40] <falcojr> hamalq: commented on the PR
[00:44] <hamalq> how is that change different than what i already changed
[00:45] <hamalq> ```if util.get_cfg_option_bool(self._cfg, "prefer_fqdn_over_hostname",` will not have the CFG since it send to cloud.Cloud right
[00:59] <hamalq> falcojr: done tthe change u requested although the main ticket was suggesting rmoving the select_hostname from rhel.py and was refused
[01:05] <falcojr> That's because it changed the functionality to do something different. This doesn't do that in absence of the config option, does it?
[01:06] <falcojr> If it does, then ignore my suggestion 😁
[01:07] <hamalq> ok i will trust on u on that
[01:07] <hamalq> anyway the change is there
=== cpaelzer_ is now known as cpaelzer
=== hjensas|afk is now known as hjensas
[10:09] <mbarroso> hello all; is it possible to specify multiple data sources ? we have a use case for kubevirt where we are injecting ssh-keys over ConfigDrive metadata but would like to use NoCloud networkData (the format is a lot nicer imo).
[13:26] <smoser> mbarroso: it is not possible.
[13:29] <smoser> my first thought was that cloud-init 'net-convert' could convert from network-config-v1 or v2 (netplan) to openstack.
[13:29] <smoser> (which woudl allow you to just have it do the conversion for you)
[13:30] <smoser> but net-convert does only outputs eni,netplan,sysconfig.  its not full A<->B.
[13:44] <mbarroso> smoser: thanks :) but other than the conversion, would it be possible for cloud-init on bootup to pick up data from 2 different data sources ? e.g. get the userdata (or metadata) from one of them, and the networkdata from another ?
[16:18] <mbarroso> rharper: sorry to ping you personally, but do you happen to know if it is possible for cloud-init to pick up data from 2 different data sources on vm bootup? e.g. get the userdata (or metadata) from one of them, and the networkdata from another ?
[16:28] <blackboxsw> mbarroso: it's not really in the general architecture of cloud-init, some datasources have grown the ability to "see/collect" metadata from two different potential sources (like flat files in a /var/lib/cloud/seed directory) but generally the metadata is taken exclusively from one discovered "source" .
[16:29] <blackboxsw> So trying to merge 2 separate datasource metadata and prioritize which metadata overrides other data is probably a fairly complex feature to design in cloud-init proper.
[16:30] <blackboxsw> right now most datasources hard-code where they grab user-data, network-data and vendor-data. There isn't a configuration primitive defined that allows us to override where to grab the data "outside" of the discovered datasource logic which is unique to each datasource
[16:42] <mbarroso> blackboxsw: thanks for your reply. Guess we'll have to port the network config to the openstack format when we use that ssh injection feature. thanks again for the reply.
[17:20] <hamalq> blackboxsw: falcojr and me fixed the integration test problem can i get +1 https://github.com/canonical/cloud-init/pull/859
[17:24] <blackboxsw> hamalq: great thanks for working on this. let
[17:24] <blackboxsw> let's see if we have any loose ends here.
[17:24] <hamalq> sure thanks
[19:31] <rharper> blackboxsw: for approved PRs, do you just click the squash and merge button these days? I
[19:31] <rharper> looking at https://github.com/canonical/cloud-init/pull/866
[19:31] <rharper> just doc change, but wanted to check before I "push the button"
[19:32] <blackboxsw> rharper: I realize we hadn't doc'd that process. For I merge we manually click the squash and merge button and manually cut-n-paste the desired comment msg into the PR ensuring it's got the right line width and no leading spaces etc.
[19:33] <blackboxsw> ultimately we should instrument logic to do this for us as it's error-prone
[19:33] <blackboxsw> currently no github actions that handle a custom label etc.
[19:33] <rharper> yeah ... I miss just "push the button" / autoland ;     and the Co-authored line? keep or remove ?
[19:34] <blackboxsw> rharper: I drop the Co-authored line in almost all cases if it was inconsequential (mostly github adds it for anyone you clicked the "update branch" button)
[19:34] <rharper> ok
[19:34] <rharper> and you leave the (#XXX) in the subject ?
[19:35] <rharper> doc: Replace remaining references to user-scripts as a config module (#866)
[19:35] <blackboxsw> yep
[19:35] <rharper> ok
[19:35] <rharper> cool, thanks for the step-by-step