mirror of
https://git.yoctoproject.org/poky
synced 2026-05-30 00:20:08 +00:00
bitbake: toaster: git clone progress bar
If a project has a lot of additional layers, the build may appear to hang while those layers are checked out. This patch adds a clone progress bar that is visible before the parsing progress appears. [YOCTO #9916] (Bitbake rev: 0c94d947b74c4dee23d7b9d255facd3cf839ccbe) Signed-off-by: David Reyna <David.Reyna@windriver.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
This commit is contained in:
committed by
Richard Purdie
parent
d74bcbeaf2
commit
43aaa802c3
@@ -85,6 +85,11 @@ class LocalhostBEController(BuildEnvironmentController):
|
|||||||
return local_checkout_path
|
return local_checkout_path
|
||||||
|
|
||||||
|
|
||||||
|
def setCloneStatus(self,bitbake,status,total,current):
|
||||||
|
bitbake.req.build.repos_cloned=current
|
||||||
|
bitbake.req.build.repos_to_clone=total
|
||||||
|
bitbake.req.build.save()
|
||||||
|
|
||||||
def setLayers(self, bitbake, layers, targets):
|
def setLayers(self, bitbake, layers, targets):
|
||||||
""" a word of attention: by convention, the first layer for any build will be poky! """
|
""" a word of attention: by convention, the first layer for any build will be poky! """
|
||||||
|
|
||||||
@@ -147,7 +152,13 @@ class LocalhostBEController(BuildEnvironmentController):
|
|||||||
logger.info("Using pre-checked out source for layer %s", cached_layers)
|
logger.info("Using pre-checked out source for layer %s", cached_layers)
|
||||||
|
|
||||||
# 3. checkout the repositories
|
# 3. checkout the repositories
|
||||||
|
clone_count=0
|
||||||
|
clone_total=len(gitrepos.keys())
|
||||||
|
self.setCloneStatus(bitbake,'Started',clone_total,clone_count)
|
||||||
for giturl, commit in gitrepos.keys():
|
for giturl, commit in gitrepos.keys():
|
||||||
|
self.setCloneStatus(bitbake,'progress',clone_total,clone_count)
|
||||||
|
clone_count += 1
|
||||||
|
|
||||||
localdirname = os.path.join(self.be.sourcedir, self.getGitCloneDirectory(giturl, commit))
|
localdirname = os.path.join(self.be.sourcedir, self.getGitCloneDirectory(giturl, commit))
|
||||||
logger.debug("localhostbecontroller: giturl %s:%s checking out in current directory %s" % (giturl, commit, localdirname))
|
logger.debug("localhostbecontroller: giturl %s:%s checking out in current directory %s" % (giturl, commit, localdirname))
|
||||||
|
|
||||||
@@ -198,6 +209,7 @@ class LocalhostBEController(BuildEnvironmentController):
|
|||||||
if name != "bitbake":
|
if name != "bitbake":
|
||||||
layerlist.append(localdirpath.rstrip("/"))
|
layerlist.append(localdirpath.rstrip("/"))
|
||||||
|
|
||||||
|
self.setCloneStatus(bitbake,'complete',clone_total,clone_count)
|
||||||
logger.debug("localhostbecontroller: current layer list %s " % pformat(layerlist))
|
logger.debug("localhostbecontroller: current layer list %s " % pformat(layerlist))
|
||||||
|
|
||||||
# 5. create custom layer and add custom recipes to it
|
# 5. create custom layer and add custom recipes to it
|
||||||
|
|||||||
@@ -0,0 +1,24 @@
|
|||||||
|
# -*- coding: utf-8 -*-
|
||||||
|
from __future__ import unicode_literals
|
||||||
|
|
||||||
|
from django.db import migrations, models
|
||||||
|
|
||||||
|
class Migration(migrations.Migration):
|
||||||
|
|
||||||
|
dependencies = [
|
||||||
|
('orm', '0015_layer_local_source_dir'),
|
||||||
|
]
|
||||||
|
|
||||||
|
operations = [
|
||||||
|
migrations.AddField(
|
||||||
|
model_name='build',
|
||||||
|
name='repos_cloned',
|
||||||
|
field=models.IntegerField(default=0),
|
||||||
|
),
|
||||||
|
migrations.AddField(
|
||||||
|
model_name='build',
|
||||||
|
name='repos_to_clone',
|
||||||
|
field=models.IntegerField(default=1),
|
||||||
|
),
|
||||||
|
]
|
||||||
|
|
||||||
@@ -437,6 +437,12 @@ class Build(models.Model):
|
|||||||
# number of recipes parsed so far for this build
|
# number of recipes parsed so far for this build
|
||||||
recipes_parsed = models.IntegerField(default=0)
|
recipes_parsed = models.IntegerField(default=0)
|
||||||
|
|
||||||
|
# number of repos to clone for this build
|
||||||
|
repos_to_clone = models.IntegerField(default=1)
|
||||||
|
|
||||||
|
# number of repos cloned so far for this build
|
||||||
|
repos_cloned = models.IntegerField(default=0)
|
||||||
|
|
||||||
@staticmethod
|
@staticmethod
|
||||||
def get_recent(project=None):
|
def get_recent(project=None):
|
||||||
"""
|
"""
|
||||||
@@ -667,6 +673,13 @@ class Build(models.Model):
|
|||||||
else:
|
else:
|
||||||
return False
|
return False
|
||||||
|
|
||||||
|
def is_cloning(self):
|
||||||
|
"""
|
||||||
|
True if the build is still cloning repos
|
||||||
|
"""
|
||||||
|
return self.outcome == Build.IN_PROGRESS and \
|
||||||
|
self.repos_cloned < self.repos_to_clone
|
||||||
|
|
||||||
def is_parsing(self):
|
def is_parsing(self):
|
||||||
"""
|
"""
|
||||||
True if the build is still parsing recipes
|
True if the build is still parsing recipes
|
||||||
@@ -698,6 +711,8 @@ class Build(models.Model):
|
|||||||
return 'Cancelling';
|
return 'Cancelling';
|
||||||
elif self.is_queued():
|
elif self.is_queued():
|
||||||
return 'Queued'
|
return 'Queued'
|
||||||
|
elif self.is_cloning():
|
||||||
|
return 'Cloning'
|
||||||
elif self.is_parsing():
|
elif self.is_parsing():
|
||||||
return 'Parsing'
|
return 'Parsing'
|
||||||
elif self.is_starting():
|
elif self.is_starting():
|
||||||
|
|||||||
@@ -61,6 +61,12 @@ function mrbSectionInit(ctx){
|
|||||||
return (cached.recipes_parsed_percentage !== build.recipes_parsed_percentage);
|
return (cached.recipes_parsed_percentage !== build.recipes_parsed_percentage);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
// returns true if the number of repos cloned/to clone changed
|
||||||
|
function cloneProgressChanged(build) {
|
||||||
|
var cached = getCached(build);
|
||||||
|
return (cached.repos_cloned_percentage !== build.repos_cloned_percentage);
|
||||||
|
}
|
||||||
|
|
||||||
function refreshMostRecentBuilds(){
|
function refreshMostRecentBuilds(){
|
||||||
libtoaster.getMostRecentBuilds(
|
libtoaster.getMostRecentBuilds(
|
||||||
libtoaster.ctx.mostRecentBuildsUrl,
|
libtoaster.ctx.mostRecentBuildsUrl,
|
||||||
@@ -100,6 +106,15 @@ function mrbSectionInit(ctx){
|
|||||||
|
|
||||||
container.html(html);
|
container.html(html);
|
||||||
}
|
}
|
||||||
|
else if (cloneProgressChanged(build)) {
|
||||||
|
// update the clone progress text
|
||||||
|
selector = '#repos-cloned-percentage-' + build.id;
|
||||||
|
$(selector).html(build.repos_cloned_percentage);
|
||||||
|
|
||||||
|
// update the recipe progress bar
|
||||||
|
selector = '#repos-cloned-percentage-bar-' + build.id;
|
||||||
|
$(selector).width(build.repos_cloned_percentage + '%');
|
||||||
|
}
|
||||||
else if (tasksProgressChanged(build)) {
|
else if (tasksProgressChanged(build)) {
|
||||||
// update the task progress text
|
// update the task progress text
|
||||||
selector = '#build-pc-done-' + build.id;
|
selector = '#build-pc-done-' + build.id;
|
||||||
|
|||||||
@@ -64,7 +64,9 @@
|
|||||||
</div>
|
</div>
|
||||||
|
|
||||||
<div data-build-state="<%:state%>">
|
<div data-build-state="<%:state%>">
|
||||||
<%if state == 'Parsing'%>
|
<%if state == 'Cloning'%>
|
||||||
|
<%include tmpl='#cloning-repos-build-template'/%>
|
||||||
|
<%else state == 'Parsing'%>
|
||||||
<%include tmpl='#parsing-recipes-build-template'/%>
|
<%include tmpl='#parsing-recipes-build-template'/%>
|
||||||
<%else state == 'Queued'%>
|
<%else state == 'Queued'%>
|
||||||
<%include tmpl='#queued-build-template'/%>
|
<%include tmpl='#queued-build-template'/%>
|
||||||
@@ -98,6 +100,31 @@
|
|||||||
</div>
|
</div>
|
||||||
</script>
|
</script>
|
||||||
|
|
||||||
|
<!-- cloning repos build -->
|
||||||
|
<script id="cloning-repos-build-template" type="text/x-jsrender">
|
||||||
|
<!-- progress bar and parse completion percentage -->
|
||||||
|
<div data-role="build-status" class="col-md-4 col-md-offset-1 progress-info">
|
||||||
|
<!-- progress bar -->
|
||||||
|
<div class="progress">
|
||||||
|
<div id="repos-cloned-percentage-bar-<%:id%>"
|
||||||
|
style="width: <%:repos_cloned_percentage%>%;"
|
||||||
|
class="progress-bar">
|
||||||
|
</div>
|
||||||
|
</div>
|
||||||
|
</div>
|
||||||
|
|
||||||
|
<div class="col-md-4 progress-info">
|
||||||
|
<!-- parse completion percentage -->
|
||||||
|
<span class="glyphicon glyphicon-question-sign get-help get-help-blue"
|
||||||
|
title="Toaster is cloning the repos required for your build">
|
||||||
|
</span>
|
||||||
|
|
||||||
|
Cloning <span id="repos-cloned-percentage-<%:id%>"><%:repos_cloned_percentage%></span>% complete
|
||||||
|
|
||||||
|
<%include tmpl='#cancel-template'/%>
|
||||||
|
</div>
|
||||||
|
</script>
|
||||||
|
|
||||||
<!-- parsing recipes build -->
|
<!-- parsing recipes build -->
|
||||||
<script id="parsing-recipes-build-template" type="text/x-jsrender">
|
<script id="parsing-recipes-build-template" type="text/x-jsrender">
|
||||||
<!-- progress bar and parse completion percentage -->
|
<!-- progress bar and parse completion percentage -->
|
||||||
|
|||||||
@@ -511,6 +511,10 @@ class MostRecentBuildsView(View):
|
|||||||
int((build_obj.recipes_parsed /
|
int((build_obj.recipes_parsed /
|
||||||
build_obj.recipes_to_parse) * 100)
|
build_obj.recipes_to_parse) * 100)
|
||||||
|
|
||||||
|
build['repos_cloned_percentage'] = \
|
||||||
|
int((build_obj.repos_cloned /
|
||||||
|
build_obj.repos_to_clone) * 100)
|
||||||
|
|
||||||
tasks_complete_percentage = 0
|
tasks_complete_percentage = 0
|
||||||
if build_obj.outcome in (Build.SUCCEEDED, Build.FAILED):
|
if build_obj.outcome in (Build.SUCCEEDED, Build.FAILED):
|
||||||
tasks_complete_percentage = 100
|
tasks_complete_percentage = 100
|
||||||
|
|||||||
Reference in New Issue
Block a user