Incorrect publish status when a build has no children
Bug #1210366 reported by
Francis Ginther
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Helipad |
Fix Released
|
High
|
Francis Ginther | ||
Bug Description
This build resulted in a published status when no publish job actually ran:
https:/
This build had no children, which isn't handled by the heuristics. Looking through prior runs, this has happened twice in the last 30 runs.
Related branches
lp:~fginther/helipad/fix_lp1210366
- PS Jenkins bot: Approve (continuous-integration)
- Chris Johnston: Approve
-
Diff: 17 lines (+6/-1)1 file modifiedutil/heuristics.py (+6/-1)
- No reviews requested
-
Diff: 61494 lines (+60935/-0) (has conflicts)105 files modified.bzrignore (+7/-0)
__init__.py (+14/-0)
admin.py (+72/-0)
api.py (+71/-0)
dashboard.py (+41/-0)
fixtures/duplicate_job_fixture.json (+1196/-0)
fixtures/head-indicators.json (+24885/-0)
fixtures/phablet-apps.json (+2158/-0)
management/__init__.py (+14/-0)
management/commands/__init__.py (+14/-0)
management/commands/jenkins_pull_ci.py (+179/-0)
migrations/0001_initial.py (+250/-0)
migrations/__init__.py (+14/-0)
models.py (+493/-0)
static/ci/css/ci_style.css (+22/-0)
static/ci/js/gauge.js (+282/-0)
templates/ci/ci_layout.html (+2/-0)
templates/ci/daily_release_overview.html (+75/-0)
templates/ci/job.html (+65/-0)
templates/ci/project_detail.html (+130/-0)
templates/ci/project_list.html (+153/-0)
templates/ci/stack_list.html (+72/-0)
templatetags/__init__.py (+14/-0)
templatetags/ci_extras.py (+40/-0)
templatetags/percentage.py (+29/-0)
tests/__init__.py (+19/-0)
tests/data/compiz-mbs-autolanding-193-armhf.json (+1/-0)
tests/data/compiz-mbs-autolanding-193.json (+1/-0)
tests/data/compiz-mbs-autolanding.json (+1/-0)
tests/data/nux-mbs-autolanding-98-amd64-testReport.json (+1/-0)
tests/data/nux-mbs-autolanding-98-amd64.coverage.xml (+5494/-0)
tests/data/nux-mbs-autolanding-98-amd64.json (+1/-0)
tests/data/nux-mbs-autolanding-98-armhf.coverage.xml (+5483/-0)
tests/data/nux-mbs-autolanding-98-armhf.json (+1/-0)
tests/data/nux-mbs-autolanding-98-i386.coverage.xml (+5494/-0)
tests/data/nux-mbs-autolanding-98-i386.json (+1/-0)
tests/data/nux-mbs-autolanding-98.json (+1/-0)
tests/data/nux-mbs-autolanding.json (+1/-0)
tests/data/ubuntu-ui-toolkit-autolanding.coverage.xml (+9076/-0)
tests/extractor/__init__.py (+22/-0)
tests/extractor/test_extractor.py (+135/-0)
tests/extractor/test_jenkins_build.py (+109/-0)
tests/extractor/test_jenkins_coverage.py (+208/-0)
tests/extractor/test_jenkins_job.py (+78/-0)
tests/extractor/test_jenkins_run.py (+151/-0)
tests/extractor/test_jenkins_test_report.py (+67/-0)
tests/extractor/utilities.py (+46/-0)
tests/functional/__init__.py (+14/-0)
tests/functional/default_page/__init__.py (+14/-0)
tests/functional/default_page/test_footer_exists.py (+21/-0)
tests/functional/default_page/test_title_contains_upstream.py (+20/-0)
tests/functional/default_page/test_top_nav.py (+20/-0)
tests/functional/projects/__init__.py (+14/-0)
tests/functional/projects/test_goto_projects.py (+22/-0)
tests/functional/projects/test_project_footer_exists.py (+23/-0)
tests/functional/projects/test_project_table_headers.py (+31/-0)
tests/functional/shared/__init__.py (+14/-0)
tests/functional/shared/settings.py (+16/-0)
tests/functional/stacks/__init__.py (+14/-0)
tests/functional/stacks/test_goto_stacks.py (+22/-0)
tests/functional/stacks/test_stack_footer_exists.py (+23/-0)
tests/functional/stacks/test_stack_table_headers.py (+28/-0)
tests/integration/__init__.py (+22/-0)
tests/integration/test_django_jenkins_build.py (+46/-0)
tests/integration/test_jenkins_build.py (+138/-0)
tests/integration/test_jenkins_job.py (+109/-0)
tests/integration/test_view_build.py (+41/-0)
tests/integration/test_view_daily_release_overview.py (+20/-0)
tests/integration/test_view_index.py (+24/-0)
tests/integration/test_view_stack.py (+35/-0)
tests/test_jenkins_pull_ci.py (+191/-0)
tests/unit/__init__.py (+34/-0)
tests/unit/test_build.py (+41/-0)
tests/unit/test_ci_project.py (+171/-0)
tests/unit/test_ci_project_import.py (+56/-0)
tests/unit/test_cu2d_stack.py (+25/-0)
tests/unit/test_cu2d_stack_import.py (+57/-0)
tests/unit/test_heuristics.py (+165/-0)
tests/unit/test_jenkins_build.py (+94/-0)
tests/unit/test_jenkins_build_import.py (+266/-0)
tests/unit/test_jenkins_coverage_report.py (+40/-0)
tests/unit/test_jenkins_coverage_report_import.py (+72/-0)
tests/unit/test_jenkins_flavor.py (+26/-0)
tests/unit/test_jenkins_job.py (+171/-0)
tests/unit/test_jenkins_job_import.py (+219/-0)
tests/unit/test_jenkins_result.py (+26/-0)
tests/unit/test_jenkins_test_report_import.py (+45/-0)
tests/unit/test_launchpad_project.py (+26/-0)
tests/unit/test_launchpad_project_import.py (+57/-0)
tests/unit/test_release.py (+26/-0)
tests/unit/test_series.py (+26/-0)
urls_api.py (+26/-0)
urls_daily.py (+28/-0)
urls_merger.py (+30/-0)
util/__init__.py (+14/-0)
util/ci_project.py (+65/-0)
util/coverage_report.py (+36/-0)
util/cu2d_stack.py (+72/-0)
util/extractor.py (+676/-0)
util/heuristics.py (+134/-0)
util/jenkins_build.py (+268/-0)
util/jenkins_job.py (+145/-0)
util/jenkins_test_report.py (+27/-0)
util/launchpad_project.py (+26/-0)
views.py (+140/-0)
| Changed in helipad: | |
| importance: | Undecided → High |
| status: | New → Confirmed |
Revision history for this message
| Francis Ginther (fginther) wrote | #1 |
Revision history for this message
| Francis Ginther (fginther) wrote | #2 |
Revision history for this message
| PS Jenkins bot (ps-jenkins) wrote | #3 |
| Changed in helipad: | |
| status: | Confirmed → Fix Committed |
| Changed in helipad: | |
| status: | Fix Committed → Fix Released |
| assignee: | nobody → Francis Ginther (fginther) |
To post a comment you must log in.
Before fixing, I've asked jibel and didrocks for an explanation and the correct reporting status.