Thursday, 2018-07-26

*** rcernin has joined #openstack-sahara00:03
*** ukaynar has quit IRC01:09
*** ukaynar has joined #openstack-sahara01:10
*** ukaynar has quit IRC01:10
*** ukaynar has joined #openstack-sahara01:11
*** ukaynar has quit IRC01:11
*** ukaynar has joined #openstack-sahara01:12
*** ukaynar has quit IRC01:16
*** openstack has joined #openstack-sahara02:50
*** ChanServ sets mode: +o openstack02:50
*** dave-mcc_ has quit IRC03:02
openstackgerritMerged openstack/sahara master: Enable mutable config in sahara  https://review.openstack.org/58570804:22
openstackgerritMerged openstack/python-saharaclient master: Clean S3 data source create/update  https://review.openstack.org/58264304:22
*** pgadiya has joined #openstack-sahara05:35
*** pgadiya has quit IRC05:35
*** links has joined #openstack-sahara05:48
*** hoonetorg has quit IRC06:57
*** hoonetorg has joined #openstack-sahara07:13
*** pcaruana has joined #openstack-sahara07:25
*** tosky has joined #openstack-sahara07:38
*** Bhujay has joined #openstack-sahara08:14
openstackgerritMerged openstack/sahara-dashboard master: APIv2 in dashboard  https://review.openstack.org/58187808:16
openstackgerritMerged openstack/sahara-dashboard master: Support of S3 data sources in dashboard  https://review.openstack.org/58074808:28
*** Bhujay has quit IRC08:37
*** Bhujay has joined #openstack-sahara08:37
*** tesseract has joined #openstack-sahara08:42
openstackgerritMerged openstack/python-saharaclient master: Support of S3 data sources in OSC  https://review.openstack.org/58072709:24
*** Bhujay has quit IRC09:30
*** Bhujay has joined #openstack-sahara09:30
*** blkart has joined #openstack-sahara09:47
*** Bhujay has quit IRC09:48
*** Bhujay has joined #openstack-sahara09:48
*** Bhujay has quit IRC09:51
*** Bhujay has joined #openstack-sahara09:52
*** Bhujay has quit IRC09:54
*** Bhujay has joined #openstack-sahara09:55
*** Bhujay has quit IRC09:57
*** Bhujay has joined #openstack-sahara09:58
blkartanybody is here? I have one question: when i create new node group template on dashboard, i found that if i select "yarn_RESOURCEMANAGER", the "SPARK_ON_YARN Parameters" label will be shown on. i don't understand why "SPARK_ON_YARN Parameters" label shown on at this time. Anybody can help me? Thanks a lot.10:03
blkartthis is the snapshot https://postimg.cc/image/ohvwsajsh/10:03
toskyI guess a dependency, but I don't know that part of code too much10:06
toskyCDH, anyway10:06
blkarttosky,  You mean it's not dependency in sahara? That's the dependency of CDH?10:12
toskyno, probably set in sahara somehow10:14
toskyI can try to find the code, but it may take time; it's not the part of code that I know better10:15
blkartI get the cdh plugin configurations by  https://developer.openstack.org/api-ref/data-processing/v1.1/index.html#show-plugin-version-details , but i don't get anything.10:15
blkarttosky,  Thanks very much.10:16
*** Bhujay has quit IRC10:22
*** Bhujay has joined #openstack-sahara10:23
*** Bhujay has quit IRC10:56
*** Bhujay has joined #openstack-sahara10:57
*** Bhujay has quit IRC10:57
*** Bhujay has joined #openstack-sahara10:58
*** Bhujay has quit IRC10:59
*** Bhujay has joined #openstack-sahara10:59
*** rcernin has quit IRC11:04
*** dave-mccowan has joined #openstack-sahara11:07
*** Bhujay has quit IRC11:11
*** Bhujay has joined #openstack-sahara11:11
*** Bhujay has quit IRC11:27
*** Bhujay has joined #openstack-sahara11:27
*** Bhujay has quit IRC11:33
*** Bhujay has joined #openstack-sahara11:38
*** Bhujay has quit IRC12:00
openstackgerritTelles Mota Vidal Nóbrega proposed openstack/python-saharaclient master: Preparing OSC for APIv2  https://review.openstack.org/58411012:14
tellesnobregatosky, we are almost done for the release today12:31
openstackgerritTelles Mota Vidal Nóbrega proposed openstack/sahara-dashboard master: Boot from volume  https://review.openstack.org/57522712:33
*** openstackgerrit has quit IRC12:36
*** links has quit IRC12:46
toskytellesnobrega: almost - do you consider 575227 as critical for the release? (there are unit test failures)13:27
tellesnobregayes, I will fix it13:32
tellesnobregaunit test?13:32
tellesnobregatosky, I see integration failure, but not unit13:34
tellesnobregaI think you meant 58411013:36
tellesnobregaI'm working on it13:36
toskyups, sorry, right13:41
toskymessed up13:41
*** Bhujay has joined #openstack-sahara13:47
*** Bhujay has quit IRC13:47
tellesnobregaI think we can skip meeting today13:53
tellesnobreganot much to update13:53
toskyoki13:58
*** zchkun has joined #openstack-sahara13:58
*** zchkun has quit IRC14:27
tellesnobregatosky, I need help to fix the unit failure14:48
tellesnobregawhat changed is right now the mock returns ids, but we need cluster objects14:48
tellesnobregawhat I'm thinking is I need to mock the clusters and return those on the mock of the creation14:51
*** pcaruana has quit IRC15:38
toskytellesnobrega: ups, sorry15:41
toskytellesnobrega: is the same failure for all 3 tests, or the two tests about data sources are failing for a different reason?15:42
*** tesseract has quit IRC16:39
tellesnobregatosky, not sure about data sources failures17:32
tellesnobregathe data sources could be just a rebase issue17:33
toskytellesnobrega: if you rebase your patch, you will see them also locally; they are easily fixed17:33
tellesnobregaok17:33
toskyI'm not sure about the cluster one, I'm trying to follow the workflow17:33
tellesnobregathe problem is just with the test_cluster_create_with_count17:34
tellesnobregaif you see the problem is with the mock object17:34
tellesnobregawhere it sets to return a dict with cluster_ids on it17:35
tellesnobregaon api v2 it should be a dict with clusters on it17:35
toskyuh17:39
tellesnobregaI would need to mock cluster objects on for the return of the creation17:42
tellesnobreganot sure how to do that17:42
toskyuhm, create fake cluster objects with mock, with just the minimum amount of properties, and change the value assigned with clusters_mock.to_dict.return_value17:44
toskyI'm trying to see how this fake list of clusters should be, but I get other errors on the way (see my comments on the review)17:57
toskyand I'm not going to be available soon for the rest of the evening17:57
tellesnobregaok, we need to hurry them17:58
tellesnobregaI guess we will have to postpone this one17:59
tellesnobregathe issue you had with the plugin version being empty17:59
tellesnobregaI saw that with cluster list as well17:59
tellesnobregabut once I created a new cluster template using api v217:59
tellesnobregait showed up17:59
tellesnobregait could be just not finding because it was created with api v118:00
toskyuh, that's interesting18:00
toskybecause the field in fact was changed18:00
tellesnobregait could be the same issue18:00
toskybut then, how is the right way to handle those changes? Should the operation done with different API version lead to the same result, if you are working with data?18:01
*** DuncanT_ has left #openstack-sahara18:01
tellesnobregait should18:01
tellesnobregawe might need some more time to figure this one out18:01
tellesnobregasince we won't have api v2 stable18:02
toskyalso, the other error with cluster create18:02
tellesnobregayes18:02
tellesnobregathat one too18:02
tellesnobregasince you are going away18:02
tellesnobregawe might not have time18:02
toskyyeah, probably better not to rush with a half-backed thing18:02
toskywe will need some patches later to fix it anyway18:02
tellesnobregabut I will take a look18:02
tellesnobregayes18:02
toskyI won't be around tomorrow either, so... maybe time to cut the release?18:08
tellesnobregawill do it now18:14
tellesnobregatosky, https://review.openstack.org/58631118:20
tosky\o/18:54
*** openstackgerrit has joined #openstack-sahara18:55
openstackgerritMerged openstack/sahara-extra stable/rocky: Update .gitreview for stable/rocky  https://review.openstack.org/58618218:55
toskythanks, out now, available as tosky[m] on matrix.org18:57
*** tosky has quit IRC18:57
*** pcaruana has joined #openstack-sahara19:17
*** pcaruana has quit IRC19:45
*** rcernin has joined #openstack-sahara22:19
*** knikolla has quit IRC23:01

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!