00:00:04 <nitzmahone> #startmeeting Ansible Azure Working Group
00:00:04 <zodbot> Meeting started Thu Aug 16 00:00:04 2018 UTC.
00:00:04 <zodbot> This meeting is logged and archived in a public location.
00:00:04 <zodbot> The chair is nitzmahone. Information about MeetBot at http://wiki.debian.org/MeetBot.
00:00:04 <zodbot> Useful Commands: #action #agreed #halp #info #idea #link #topic.
00:00:04 <zodbot> The meeting name has been set to 'ansible_azure_working_group'
00:00:22 <nitzmahone> #chair yungezz yuwei
00:00:22 <zodbot> Current chairs: nitzmahone yungezz yuwei
00:00:30 <yungezz> hello
00:00:40 <nitzmahone> howdy
00:00:56 <Kylie_> Hi all
00:00:59 <Kylie_> Who is online?
00:01:08 <nitzmahone> #chair Kylie_
00:01:08 <zodbot> Current chairs: Kylie_ nitzmahone yungezz yuwei
00:01:41 <zodbot> Kylie_: Error: Can't start another meeting, one is in progress.
00:01:52 * nitzmahone already started it
00:01:59 <Kylie_> O, sorry, miss the message for "startmeeting".
00:02:14 <nitzmahone> #chair zikalino82
00:02:14 <zodbot> Current chairs: Kylie_ nitzmahone yungezz yuwei zikalino82
00:02:32 <Kylie_> Today's topic is to discuss PRs for Ansible 2.7.
00:02:40 <Kylie_> Catherine sent out the list today.
00:03:02 <nitzmahone> Yep, saw that come through this morning
00:03:07 <Kylie_> New feature:
00:03:07 <Kylie_> Support vmss in inventory: https://github.com/ansible/ansible/pull/40327
00:03:07 <Kylie_> New modules:
00:03:07 <Kylie_> App gateway: https://github.com/ansible/ansible/pull/39940
00:03:07 <Kylie_> Traffic manager: https://github.com/ansible/ansible/pull/43812
00:03:07 <Kylie_> Vmss autoscale: https://github.com/ansible/ansible/pull/41533
00:03:07 <Kylie_> Route table: https://github.com/ansible/ansible/pull/41175
00:03:08 <Kylie_> Web_app facts: https://github.com/ansible/ansible/pull/43631
00:03:08 <Kylie_> Sql firewall rule: https://github.com/ansible/ansible/pull/34615
00:04:29 <Kylie_> And Matt, how about inventory performance tuning work?
00:04:31 <nitzmahone> We'll review ASAP- core freeze is next Thursday, so we're all heads-down trying to get our own stuff in for that as well
00:04:55 <yuwei> next Thursday?
00:04:59 <nitzmahone> Working on the inventory plugin today; have both the batch and fallback serial modes working
00:05:06 <nitzmahone> (core freeze, community freeze is one week later)
00:05:57 <yuwei> yes
00:06:07 <Kylie_> When you say inventory plugin, does it mean improving the performance? Will it be in 2.7?
00:06:10 <nitzmahone> I wish I could get a statement of support on the batch API endpoint, but since we can't, we have to implement a fallback mode to get things one at a time
00:06:13 <nitzmahone> Yes
00:06:38 <nitzmahone> It's a completely new implementation in a different form that supersedes the inventory script
00:07:05 <nitzmahone> (allows, among other things, for code to be shared between the modules/lookups/etc and the inventory stuff)
00:08:17 <Kylie_> Nice. Can it catch up 2.7?
00:08:31 <nitzmahone> It will be in 2.7
00:08:55 <Kylie_> Great. Noted.
00:09:05 <yungezz> Will vmss added after the batch inventory out?
00:09:09 <Kylie_> Catherine, will you have one module for CDN?
00:09:19 <yungezz> Yes
00:09:29 <nitzmahone> New inventory will support VMSS from the start
00:09:54 <yungezz> Actually 3,seperate cdn profile endpoints custom domain
00:09:59 <yungezz> Great
00:10:26 <nitzmahone> When will the CDN PRs hit?
00:10:50 <yungezz> Today we did test and review in role now
00:11:02 <yungezz> I mean today
00:11:39 <nitzmahone> Hmm, seems like not going to be any significant real-world exposure/usage on that by the time 2.7 freeze comes.
00:12:04 <nitzmahone> The point of putting things in the role was to get real-world validation *before* they're submitted for inclusion
00:12:20 <nitzmahone> Not concurrently with submission
00:13:22 <nitzmahone> I haven't been watching the role- have any of this latest batch of modules received any customer usage or feedback?
00:13:51 <Kylie_> Here we don't treat CDN as an independent module but mostly required by Web App scenario.
00:13:54 * nitzmahone is especially suspicious of appgateway- pretty sure there are still a number of things broken in SSL endpoints
00:14:45 <zikalino82> i have added missing tests in a meanwhile
00:15:11 <nitzmahone> Do they include configuring SSL endpoints? Last time I looked, there was no coverage on that at all
00:15:26 <zikalino82> yes
00:15:45 <nitzmahone> Cool- looking forward to checking that out
00:16:03 <zikalino82> now test includes:
00:16:05 <zikalino82> ssl_policy:
00:16:05 <zikalino82> jcm9zb2Z0IElUMR4wHAYDVQQDExVNaWNyb3NvZnQgSVQgVExTIENBIDUwHhcNMTcwNzIwMTc0NzA4WhcNMTkwNzEwMTc0NzA4WjAXMRUwEwYDVQQDEwx3d3cuYmluZy5jb20wggEiMA0GCSqGSIb3DQEBAQUAA4IBDwAwggEKAoIBAQC6jsg+/7DlIrdgFOcaDlK3RQ9sIgkJsgpj+ZxAbIe3ziyimIxjVlHX87pqgXcNhaYNbCFD0iPm+aUfbv4GDTLR+AIr8eSegqxZ+CBToYM67NhpVYra1KAvY4XgqxorO4FB9IWYJRqhI3SZeZ3lLK5t9XuUMicG8l52nJfpPdXXvBca2
00:16:06 <zikalino82> wUCq8FHEObG81vJzESA0htLLPTjdUWBQnXPiW5bqzlGHzzv8ISV6jtDLNNa5JRlhSlXho+6pCedhNF7MP4yTaantPvAELLRWX13VhjgoCcRCCu0s8rxW5DuVWl2Pb2iw35MFnNWlcoVwq0AjAfGA+xEba/WLid6qfkQctYjAgMBAAGjggfQMIIHzDAdBgNVHQ4EFgQUCYflhSl4MCAls91+3GztpSmoA3AwCwYDVR0PBAQDAgSwMB8GA1UdIwQYMBaAFAj+JZ906ocEwry7jqg4XzPG0WxlMIGsBgNVHR8EgaQwgaEwgZ6ggZuggZiGS2h0dHA6Ly9tc2NybC5taWNyb3NvZnQ
00:16:06 <zikalino82> uY29tL3BraS9tc2NvcnAvY3JsL01pY3Jvc29mdCUyMElUJTIwVExTJTIwQ0ElMjA1LmNybIZJaHR0cDovL2NybC5taWNyb3NvZnQuY29tL3BraS9tc2NvcnAvY3JsL01pY3Jvc29mdCUyMElUJTIwVExTJTIwQ0ElMjA1LmNybDCBhQYIKwYBBQUHAQEEeTB3MFEGCCsGAQUFBzAChkVodHRwOi8vd3d3Lm1pY3Jvc29mdC5jb20vcGtpL21zY29ycC9NaWNyb3NvZnQlMjBJVCUyMFRMUyUyMENBJTIwNS5jcnQwIgYIKwYBBQUHMAGGFmh0dHA6Ly9vY3NwLm1zb2NzcC5jb
00:16:07 <zikalino82> 20wPgYJKwYBBAGCNxUHBDEwLwYnKwYBBAGCNxUIh9qGdYPu2QGCyYUbgbWeYYX062CBXYTS30KC55N6AgFkAgEQMB0GA1UdJQQWMBQGCCsGAQUFBwMCBggrBgEFBQcDATBNBgNVHSAERjBEMEIGCSsGAQQBgjcqATA1MDMGCCsGAQUFBwIBFidodHRwOi8vd3d3Lm1pY3Jvc29mdC5jb20vcGtpL21zY29ycC9jcHMwJwYJKwYBBAGCNxUKBBowGDAKBggrBgEFBQcDAjAKBggrBgEFBQcDATCCBW0GA1UdEQSCBWQwggVgggx3d3cuYmluZy5jb22CEGRpY3QuYmluZy5jb20
00:16:07 <zikalino82> uY26CEyoucGxhdGZvcm0uYmluZy5jb22CCiouYmluZy5jb22CCGJpbmcuY29tghZpZW9ubGluZS5taWNyb3NvZnQuY29tghMqLndpbmRvd3NzZWFyY2guY29tghljbi5pZW9ubGluZS5taWNyb3NvZnQuY29tghEqLm9yaWdpbi5iaW5nLmNvbYINKi5tbS5iaW5nLm5ldIIOKi5hcGkuYmluZy5jb22CGGVjbi5kZXYudmlydHVhbGVhcnRoLm5ldIINKi5jbi5iaW5nLm5ldIINKi5jbi5iaW5nLmNvbYIQc3NsLWFwaS5iaW5nLmNvbYIQc3NsLWFwaS5iaW5nLm5ldIIOK
00:16:08 <zikalino82> i5hcGkuYmluZy5uZXSCDiouYmluZ2FwaXMuY29tgg9iaW5nc2FuZGJveC5jb22CFmZlZWRiYWNrLm1pY3Jvc29mdC5jb22CG2luc2VydG1lZGlhLmJpbmcub2ZmaWNlLm5ldIIOci5iYXQuYmluZy5jb22CECouci5iYXQuYmluZy5jb22CEiouZGljdC5iaW5nLmNvbS5jboIPKi5kaWN0LmJpbmcuY29tgg4qLnNzbC5iaW5nLmNvbYIQKi5hcHBleC5iaW5nLmNvbYIWKi5wbGF0Zm9ybS5jbi5iaW5nLmNvbYINd3AubS5iaW5nLmNvbYIMKi5tLmJpbmcuY29tgg9nbG9
00:16:08 <zikalino82> iYWwuYmluZy5jb22CEXdpbmRvd3NzZWFyY2guY29tgg5zZWFyY2gubXNuLmNvbYIRKi5iaW5nc2FuZGJveC5jb22CGSouYXBpLnRpbGVzLmRpdHUubGl2ZS5jb22CDyouZGl0dS5saXZlLmNvbYIYKi50MC50aWxlcy5kaXR1LmxpdmUuY29tghgqLnQxLnRpbGVzLmRpdHUubGl2ZS5jb22CGCoudDIudGlsZXMuZGl0dS5saXZlLmNvbYIYKi50My50aWxlcy5kaXR1LmxpdmUuY29tghUqLnRpbGVzLmRpdHUubGl2ZS5jb22CCzNkLmxpdmUuY29tghNhcGkuc2VhcmNoL
00:16:09 <zikalino82> mxpdmUuY29tghRiZXRhLnNlYXJjaC5saXZlLmNvbYIVY253ZWIuc2VhcmNoLmxpdmUuY29tggxkZXYubGl2ZS5jb22CDWRpdHUubGl2ZS5jb22CEWZhcmVjYXN0LmxpdmUuY29tgg5pbWFnZS5saXZlLmNvbYIPaW1hZ2VzLmxpdmUuY29tghFsb2NhbC5saXZlLmNvbS5hdYIUbG9jYWxzZWFyY2gubGl2ZS5jb22CFGxzNGQuc2VhcmNoLmxpdmUuY29tgg1tYWlsLmxpdmUuY29tghFtYXBpbmRpYS5saXZlLmNvbYIObG9jYWwubGl2ZS5jb22CDW1hcHMubGl2ZS5jb22
00:16:09 <zikalino82> CEG1hcHMubGl2ZS5jb20uYXWCD21pbmRpYS5saXZlLmNvbYINbmV3cy5saXZlLmNvbYIcb3JpZ2luLmNud2ViLnNlYXJjaC5saXZlLmNvbYIWcHJldmlldy5sb2NhbC5saXZlLmNvbYIPc2VhcmNoLmxpdmUuY29tghJ0ZXN0Lm1hcHMubGl2ZS5jb22CDnZpZGVvLmxpdmUuY29tgg92aWRlb3MubGl2ZS5jb22CFXZpcnR1YWxlYXJ0aC5saXZlLmNvbYIMd2FwLmxpdmUuY29tghJ3ZWJtYXN0ZXIubGl2ZS5jb22CE3dlYm1hc3RlcnMubGl2ZS5jb22CFXd3dy5sb2Nhb
00:16:10 <zikalino82> C5saXZlLmNvbS5hdYIUd3d3Lm1hcHMubGl2ZS5jb20uYXUwDQYJKoZIhvcNAQELBQADggIBADTpW/UWeupk40OP6k4yxihKStswxwqPAfMRmx4XyqmTAawAKRNM+6EZth1BQdPdOplwRTvs69kkmUHJH+ZjYXBezEACWkzEiNUQnzkRWajdSQIz08Ubj/mBD6U8xLYD+NXgiB0xNWabd8aiPsqPaj6I3qkNw4JvtgtHZQG1zlwC5/Lu6yV3DM3sKpQMyBmOnX6nVUiS0MTOzLgZOQzRk07nO7EXWGcKTmDBjE8cqv5IA/jQ6gtaxCI5pDxfXK4ct7oQyoChfxOXcEDKMmMndFm
00:16:10 <zikalino82> g9ch5c4an/FRM2cgzDfjR01A71LNUpLUdOjNV0T+ZEStqEpdyDFfjrHGDtzLyqEz3iyvvQFyjmlGh6OtZXwjCPpnVSrKCmfJKio0kUxyq+6t5tZAQbPVgFKiMrVnU+sgvmNVip1toijyz8vMVCkwJ2G++7xjJukoELMxZ50W4/SAMZLy1Asx02NBwYCu9+CTQPVnmPe7rmxhlQRBOfDNa1+5jwRHY64YudEzKhWR1uqS3ABd/fk+TL86yuNYGAgxnOm1FtOGieRgViV3+NzC+bDbuUOtmbD/GvDGmRwJRcCTHL7jBmkHePh2ABY93NE/IbkaDP6l1Kw98AfqkzSUxhqHXuThe7
00:16:11 <zikalino82> KIoX9/0zv4AA1WZFis1QvAG7dpl9eio6vCdC/73HvBAlqRL+7Mb1uu0
00:16:13 <zikalino82> qAUX7gXKK0ISr6Dto71rFdza0uPwbcQoPN1QwkJHSAkxWg4OcAx2kf7077tlHhf5rzxTn5V3wXM0Q3h6NsDjSqSHjYVISIiXItUzlDaBpMY/NUFNCTyJR5I91MWsljrJ/bQaAIo57HJR9nzjY5DaBA9P3bAhmX5LJRGsJWoCEaGeeVQ3Yn6yD06ordiJnf6dNxqQGN+o2x54gqfmw+RnoC2f8VAsTIfb3fwJPKdg2JiJIa6Ms2Sc8VR7VGmZt34qZwTPBrzeqJjtIMT41bBae46lmma8ypYwErqzOYSrHqXPXzaxlloYy81HYWAsJTWyBxTsVBcLom5m9ru79+SKG35xY1wSkz
00:16:14 <zikalino82> ZmWMNFfVRFCJy/X+h2ErrGYjogCHYaIUmiosvUccwRUXGU083ul9iTcz/Dl79VBz63OFX/CnZMDTQ8ugbqpvW78pAnBU0r8MUubHciD1sJG2zmMlxCAzan6BLm9OMyhTNIbzYOjQQw99MQQys/ZeyNLqTFHTeGRfU2ewqgHjbH2PYCQfjipXSmdmsSsGxlLA9AOtwAk3QKJ77P03HRGOeXmy/I4iIHuIQuaQcjfprNR2fI36dftDo7U4gvRQHkiti+zNVqpi3/hIc2k7O8bCcMeSvfIlUvWIPUrUceZmpVPpLdcFcQbN9+1nZwiFYydOhrPnlp40rSO3RM08EmQUfRYt8fwRFc
00:16:14 <zikalino82> oWBX3b411vOqZVGeMfMtThMYI53R4Cmh5tUp93FslHNmIfnuewhHfIm+vtCicLcW6TaC2l4EqmNf0flK5m5nANotCfqj87MPsB83qPwol/91BTKaxuH2hKrZDgU1ibPE8NhzBinp2ANi0BHK3Sl0CsC2MPyZpFY+4MWvk/SI9ex4VsKYKmhubOFkhDLLBZH0UEmUdNTH4Gd76GsDnfI9arR2ctM9ecTPeu74hKiHlNZhc4U3TX20FBeqF5tZYnfCLRhvdiNM9AlwEKqqQEe0W7PrALcNVdjhJl0X9+0Br28E3RKZQRITWa10Vjmh0WcYrzEQ3/qEZYbqVpHMp+kdrHxB65v0zl
00:16:15 <zikalino82> GxjdwyKzafLzqYXmaHOyVlFnkayNaAkVVxOCzNrxB9HfhjvhjWafeMvA0p7O9CxTD2xPEhUaHQ5j7L8F0alfMYcg73SdGHAcY6AV8+eh0jqs3IF68cquXOl5Bm8uYKRjtgl9nY6hYc0lRDdtFHZo8ayNDr0cltNU7XZTaCKVNSDTRn92rTNJY0E3PD5HSKcRi58WJrIgEDGasyleRkRlGTY7512Qut0rg7m1Eyp6MK+sNmSSA7cR70pH7I1dwy4VrJMODdMH11y1QJF2EQWQdN00Js54tjVgTIO3btb5N7jhNYpRedv0a4UZ8TdDI4ZMCMf3SdP3xbQ06M1pFrS8WQzwp3KTk8
00:16:15 <zikalino82> vmnseJL84n0hC8KqWmGmTWHTa9dwmopeM6Xh/Jm1pkrgrloxqfSlscGEJE0plAnk1mLx29FxswfZ6a7pNKg7CydK4SiDkqM+pWukPbgKODqYPUvS0nk3RGGXvZSIzTbvm77tF+MqXOb6Rn+IflIk4yZsjIBQA0I/bQ78YDzXUVsrtAi9waRoCZs+L48NUy4zpKn25FMqkrziVn+TGB4jANBgkrBgEEAYI3EQIxADATBgkqhkiG9w0BCRUxBgQEAQAAADBdBgkqhkiG9w0BCRQxUB5OAHQAZQAtAGMANwBmADEAYwBhADYAMQAtADQAOAA1ADQALQA0ADgAZQBmAC0AYQAwADgA
00:16:16 <zikalino82> NQAtAGQANABhADIAZgA1AGYAOAAyADcAZAAxMF0GCSsGAQQBgjcRATFQHk4ATQBpAGMAcgBvAHMAbwBmAHQAIABTAHQAcgBvAG4AZwAgAEMAcgB5AHAAdABvAGcAcgBhAHAAaABpAGMAIABQAHIAbwB2AGkAZABlAHIwggQ/BgkqhkiG9w0BBwagggQwMIIELAIBADCCBCUGCSqGSIb3DQEHATAcBgoqhkiG9w0BDAEDMA4ECFcAfrkm3ibUAgIH0ICCA/hlBog8GY9GCpucTwAxBGa0cOGQ29EK0xfrmY/Dv59IeJhRr47Mvl1XNk5PIInb64RsOVr00jrJAbfgLLKMBowcQL
00:16:16 <zikalino82> T7k6jGbGBdOzC57x9DNP0VuHsIIym0Z+SpJgEWBbtdBTGzgNw/YoXYxT4Rtka9ScSyCFjmmCzXz7bGLqC7yrGb7BzigQ9y4u4bg0pf75pERzN8rJM29Ob2IydkgARfpmbNKjdMCtD6dI7tafG2lQfUX6sgQY+Sy5HTz3ansN8X1yv2WQTu8Drxf2ce55v4WrFbPTTND94ubgDt7jvbCe1DuNP1DAYmQ5pbW0GGqF1x2csK5WWD7J8FD08VaQFM8y8pGIUeUkN4rYU3eTdTAQe+ec2hOr9QZn1Sb/p5u4KqIMn4MSCQ8EU0gXa2JETdUjXPr/5JFZTidJYagRyMIkYnwg9uusik
00:16:17 <zikalino82> ctulaBsHMBKMYQ0Z19CEbcd2phdoxWTrtp7kwwjnu64zPgE6ALe9yJOT8AFEB6H1c16Z+aPGj9hbhkh6tcdGWUvzDYq08wjKjP3nA78StIisUmeZPfAXJUquPzRZr0pmcwYWfyP54TdC2BvPlLW/QXVV44IGxUdLuI6mz4p+O2+xKu9QMFwdcpij2ZK4uMrBLDo7ZoTQ4rBRnn471AMUKgeP0D5tbl8PygUU1RqHv34ok3fwx0WglzdMQJyt53PiPWW4lipwUtUfd0eD8CXoMccf8XJmugVUBCD1wQsyCW6RrR9RX8HXVBrm5O2HKfJcQYznl3qHqXb6ofvbOQ3S+v0ALN+sma
00:16:17 <zikalino82> 8Tn6JceVTAOH+UuMdcu0FIDYnpmrvMecnJ2kbs1Y35mj4rSJyP5PGLg+ygb9VlBPwCCem/jHL+YivN38+0oWqfn2slyI4FNKX+5U8M6xpiEaq6McKwKZC1d51A4dUdMAkO2d1Z6rVjqhKeqE6HWD9A0cyPBFZpNQskUfNDW8qILLEfEjhBi+s1LkHzKDykCN/ReFfRiQS84DekoC59cymM8Hs1geMWCMFWfut4HTd7ItYaiJz5qpYVY4U/8myhyWnrktjLjQ6OkdM9bBDIpRHj95MYEC26NlWQZwjk+yynPTZf0w5p2Ok6Dq2shJFykuJ1VkelgvsPe8qMa55Wp11dpREIFzXo
00:16:18 <zikalino82> uPXNP/vFpziZcl1OKTd7Dwa+ruQFRfsoZGzupsBOYxrmYqr6kOm5mzMW0HAlsWwl5mY2aSQMvXBE6k32xTkPIlIp763Ee4m6cmx4+SDcO5D+a9t05QY4JmssL+x3T9qsbXSXDPEsg0cfVvuQYy9AYkIFOes4G45IagRAvhQQj9bEh8kTvp8CFDDtIrbWjX50zreb51VcAcEkIOLyROtIdLem0zA7MB8wBwYFKw4DAhoEFC0i4I5iwNYQug0vTVS0JC/Qm+/NBBTsUM0D9QxIZYUi+qlDy14sOcEaUwICB9A=
00:16:19 <zikalino82> haha, didn't realise it was so large :-)
00:17:22 <zikalino82> also we have additional contribution to app gateway in preview modules. so seems like somebody has started using it already
00:18:05 <nitzmahone> OK, we'll get to those reviews as soon as we can. Anything else to discuss today?
00:19:09 <Kylie_> That is what we need your support. Fully understand you also need to work on core but also only 2 weeks for our feature freeze. We really want to have some complete E2E stories in 2.7. Thank you in advance!
00:19:16 <yungezz> zikalino82: zim do you want to talk about facts module?
00:19:21 <Kylie_> #action Matt, Jordan, review below PR ASAP. Thank you.
00:19:40 <Kylie_> App gateway: https://github.com/ansible/ansible/pull/39940
00:19:40 <Kylie_> Traffic manager: https://github.com/ansible/ansible/pull/43812
00:19:40 <Kylie_> Vmss autoscale: https://github.com/ansible/ansible/pull/41533
00:19:40 <Kylie_> Route table: https://github.com/ansible/ansible/pull/41175
00:19:40 <Kylie_> Web_app facts: https://github.com/ansible/ansible/pull/43631
00:19:41 <Kylie_> Sql firewall rule: https://github.com/ansible/ansible/pull/34615
00:20:02 <zikalino82> yes, actually we are doing plenty of missing facts modules, and we were wondering with Catherine that, as they are pretty simple, perhaps we could handle merge ourselves
00:20:12 <Kylie_> As for https://github.com/ansible/ansible/pull/40327, my understanding is that it will be there when Matt's inventory work done. If not, correct me.
00:21:36 <nitzmahone> Yeah- I'm not opposed to adding it to the old inventory script either, but as there are no tests for it, verifying it's working requires a lot of time
00:23:08 <yungezz> I tried searching test for inventory but didn’t find .  I did manual test. How previous inventory pr do in this situation?
00:24:50 <nitzmahone> They don't- also one of the reasons we're trying to move to inventory plugins; they rely on a lot more shared code that has tests, and we can build tests for them easier
00:25:19 <nitzmahone> (old dynamic inventory plugins are not really "supported", hence the "contrib" directory)
00:25:35 <nitzmahone> s/plugins/scripts/
00:25:53 <nitzmahone> We're trying to move toward a model where the new ones are really supportable and testable
00:26:55 <yungezz> I tested the contributors change and committed to his branch for this vmss change
00:27:05 <yungezz> Thanks for explanation
00:27:11 <Kylie_> Once you get the PR for new inventory ready, I think we could test it w/ you together to fasten its validation this time. After 2.7 code freeze, we could discuss how to add more auto tests there.
00:27:33 <nitzmahone> That'd be great
00:29:18 <nitzmahone> As for the facts modules- yeah, I'm fine w/ merge so long as you're following the same patterns as the recent ones ("curated" return values, no "ansible_facts" in return, simple selection, proper documentation of return values)
00:29:53 <nitzmahone> Anything else for today?
00:30:50 <zikalino82> ok, thanks nitzmahone !
00:30:57 <yungezz> No from my side
00:31:02 <Kylie_> Thank you Matt. No topic from me.
00:31:10 <yungezz> Thanks Matt
00:31:22 <nitzmahone> Sounds good- be well! Talk to you next week...
00:31:25 <Kylie_> Please help review our PRs. The team will be standby to update per comments. Thank you!!
00:31:34 <Kylie_> #endmeeting