Merge lp://qastaging/~zyga/checkbox/investigation into lp://qastaging/checkbox

Proposed by Zygmunt Krynicki
Status: Rejected
Rejected by: Zygmunt Krynicki
Proposed branch: lp://qastaging/~zyga/checkbox/investigation
Merge into: lp://qastaging/checkbox
Diff against target: 509 lines (+187/-61)
10 files modified
plainbox/docs/manpages/plainbox.rst (+24/-0)
plainbox/plainbox/i18n.py (+70/-32)
plainbox/plainbox/impl/clitools.py (+21/-2)
plainbox/plainbox/impl/commands/cmd_session.py (+3/-0)
plainbox/plainbox/impl/commands/inv_session.py (+4/-1)
plainbox/plainbox/impl/session/resume.py (+4/-4)
plainbox/plainbox/impl/session/storage.py (+37/-11)
plainbox/plainbox/impl/session/test_storage.py (+5/-5)
plainbox/plainbox/impl/unit/testplan.py (+1/-1)
plainbox/plainbox/provider_manager.py (+18/-5)
To merge this branch: bzr merge lp://qastaging/~zyga/checkbox/investigation
Reviewer Review Type Date Requested Status
Zygmunt Krynicki (community) Needs Resubmitting
Sylvain Pineau (community) Approve
Review via email: mp+244126@code.qastaging.launchpad.net

Description of the change

b5ecf55 plainbox:session: respect PLAINBOX_SESSION_REPOSITORY
8229dbe plainbox:session: look for non-hidden .session directories
643a7b0 plainbox:session: fix FLAG_IGNORE_JOB_CHECKSUMS_S
9f7cfa9 plainbox:session: fix FLAG_REWRITE_LOG_PATHNAMES_F
16fbbfd plainbox:session: fix typo in comment
a2476c7 plainbox:commands: add 'session list --only-ids'

To post a comment you must log in.
Revision history for this message
Sylvain Pineau (sylvain-pineau) wrote :

I wanted PLAINBOX_SESSION_REPOSITORY, you did it. thanks a million, +1

review: Approve
Revision history for this message
Daniel Manrique (roadmr) wrote :
Download full text (11.4 KiB)

The attempt to merge lp:~zkrynicki/checkbox/investigation into lp:checkbox failed. Below is the output from the failed tests.

[precise] starting container
[precise] (timing) 0.07user 0.01system 0:04.21elapsed 1%CPU (0avgtext+0avgdata 7868maxresident)k
[precise] (timing) 0inputs+32outputs (0major+5779minor)pagefaults 0swaps
[precise] provisioning container
[precise] (timing) 37.06user 9.91system 1:21.86elapsed 57%CPU (0avgtext+0avgdata 51416maxresident)k
[precise] (timing) 0inputs+16424outputs (0major+4457123minor)pagefaults 0swaps
[precise-testing] Starting tests...
Found a test script: ./checkbox-gui/requirements/container-tests-checkbox-gui-build
[precise-testing] container-tests-checkbox-gui-build: PASS
[precise-testing] (timing) 28.45user 2.13system 0:30.96elapsed 98%CPU (0avgtext+0avgdata 116388maxresident)k
[precise-testing] (timing) 0inputs+4224outputs (0major+478243minor)pagefaults 0swaps
Found a test script: ./checkbox-ng/requirements/container-tests-checkbox-ng-unit
[precise-testing] container-tests-checkbox-ng-unit: PASS
[precise-testing] (timing) 0.54user 0.16system 0:01.70elapsed 41%CPU (0avgtext+0avgdata 39784maxresident)k
[precise-testing] (timing) 0inputs+3408outputs (0major+20447minor)pagefaults 0swaps
Found a test script: ./checkbox-support/requirements/container-tests-checkbox-support
[precise-testing] container-tests-checkbox-support: PASS
[precise-testing] (timing) 16.00user 0.16system 0:16.25elapsed 99%CPU (0avgtext+0avgdata 82816maxresident)k
[precise-testing] (timing) 0inputs+1032outputs (0major+31446minor)pagefaults 0swaps
Found a test script: ./checkbox-touch/requirements/container-tests-touch-unit-tests
[precise-testing] container-tests-touch-unit-tests: PASS
[precise-testing] (timing) 0.00user 0.00system 0:00.01elapsed 42%CPU (0avgtext+0avgdata 2020maxresident)k
[precise-testing] (timing) 0inputs+8outputs (0major+2340minor)pagefaults 0swaps
Found a test script: ./plainbox/plainbox/impl/providers/categories/requirements/container-tests-provider-categories
[precise-testing] container-tests-provider-categories: PASS
[precise-testing] (timing) 0.14user 0.04system 0:00.21elapsed 91%CPU (0avgtext+0avgdata 13452maxresident)k
[precise-testing] (timing) 0inputs+176outputs (0major+5861minor)pagefaults 0swaps
Found a test script: ./plainbox/requirements/001-container-tests-plainbox-egg-info
[precise-testing] 001-container-tests-plainbox-egg-info: PASS
[precise-testing] (timing) 0.15user 0.02system 0:00.19elapsed 92%CPU (0avgtext+0avgdata 10496maxresident)k
[precise-testing] (timing) 0inputs+88outputs (0major+4985minor)pagefaults 0swaps
Found a test script: ./plainbox/requirements/container-tests-plainbox
[precise-testing] container-tests-plainbox: FAIL
[precise-testing] stdout: http://paste.ubuntu.com/9442257/
[precise-testing] stderr: http://paste.ubuntu.com/9442258/
[precise-testing] (timing) Command exited with non-zero status 1
[precise-testing] (timing) 6.06user 0.46system 0:06.63elapsed 98%CPU (0avgtext+0avgdata 64664maxresident)k
[precise-testing] (timing) 0inputs+2640outputs (0major+52025minor)pagefaults 0swaps
Found a t...

Revision history for this message
Zygmunt Krynicki (zyga) wrote :

Bah, my bad. I'll fix this

Revision history for this message
Zygmunt Krynicki (zyga) wrote :

Fixed, I also have new fixes patches so please re-read

review: Needs Resubmitting
3488. By Zygmunt Krynicki

plainbox:session: look for non-hidden .session directories

This patch changes how plainbox discovers and enumerates sessions.
Previously any directory would be a "session" that would be inspected.
With this change only non-hidden directories that end with '.session'
are considered. This makes it friendlier to work with when storing
sessions in a different location, such as the regressions repository.

Signed-off-by: Zygmunt Krynicki <email address hidden>

3489. By Zygmunt Krynicki

plainbox:session: fix FLAG_IGNORE_JOB_CHECKSUMS_S

This patch fixes a embarrasing bug that gets triggered when session
resume is invoked with the ignore-job-checksums flag. The bug is a
result of later refactoring, which added binary flags in place of string
flags and set membership tests.

Signed-off-by: Zygmunt Krynicki <email address hidden>

3490. By Zygmunt Krynicki

plainbox:session: fix FLAG_REWRITE_LOG_PATHNAMES_F

This patch fixes a embarrasing bug that gets triggered when session
resume is invoked with the rewrite-log-pathnames flag. The bug is a
result of later refactoring, which added binary flags in place of string
flags and set membership tests.

Signed-off-by: Zygmunt Krynicki <email address hidden>

3491. By Zygmunt Krynicki

plainbox:session: fix typo in comment

Signed-off-by: Zygmunt Krynicki <email address hidden>

3492. By Zygmunt Krynicki

plainbox:commands: add 'session list --only-ids'

This patch adds a new argument to 'plainbox session list'. If invoked
with the --only-ids argument, the output is frendlier for scripting and
consists of session identifiers which are printed one per line.

Signed-off-by: Zygmunt Krynicki <email address hidden>

3493. By Zygmunt Krynicki

plainbox:units:testplan: fix get_effective_category

This patch fixes TestPlan.get_effective_category() when no overrides
are being applied and the natural category is supposed to be returned.

Signed-off-by: Zygmunt Krynicki <email address hidden>

3494. By Zygmunt Krynicki

plainbox:provider_manager: add support for i18n in manage_py_extension

Signed-off-by: Zygmunt Krynicki <email address hidden>

3495. By Zygmunt Krynicki

plainbox:provider_manager: fix PEP-8 issue

Signed-off-by: Zygmunt Krynicki <email address hidden>

3496. By Zygmunt Krynicki

plainbox:provider_manager: fail gracefully in InstallCommand.get_command_epilog()

Signed-off-by: Zygmunt Krynicki <email address hidden>

3497. By Zygmunt Krynicki

plainbox:clitools: add support for early logging

This patch adds support for early logging. Early logging can be enabled
even before command line argument parsing is attempted. It is perfect
for debugging command line handling code itself.

The feature is enabled by setting environment variables. There are three
new variables: PLAINBOX_DEBUG, PLAINBOX_TRACE and PLAINBOX_LOG_LEVEL.
The details of their behavior are documented in the plainbox manual
page.

Signed-off-by: Zygmunt Krynicki <email address hidden>

3498. By Zygmunt Krynicki

plainbox:i18n: fix non-default domain lookups

This patch changes how plainbox.i18n.{bind,}textdomain() works. In the
past they would just proxy the calls to the original python gettext
module.

This works okay for anything not using our internal i18n APIs. If, however,
someone is using dgettext() with a domain that's not 'plainbox', it would
fail to load the translation catalog from a custom domain *and* a custom
locale directory at the same time.

This patch moves textdomain and bindtextdomain functions to be a method
of the ITranslator API. Inside the GettextTranslator we now use
bindtextdomain as hints and store the locale directory which is finally
used to instantiate the translation catalog. For compatibility, both
methods are still calling their equivalents from thet gettext module.

Signed-off-by: Zygmunt Krynicki <email address hidden>

Revision history for this message
Zygmunt Krynicki (zyga) wrote :

This is now split into smaller branches so ignore this one please

Unmerged revisions

3498. By Zygmunt Krynicki

plainbox:i18n: fix non-default domain lookups

This patch changes how plainbox.i18n.{bind,}textdomain() works. In the
past they would just proxy the calls to the original python gettext
module.

This works okay for anything not using our internal i18n APIs. If, however,
someone is using dgettext() with a domain that's not 'plainbox', it would
fail to load the translation catalog from a custom domain *and* a custom
locale directory at the same time.

This patch moves textdomain and bindtextdomain functions to be a method
of the ITranslator API. Inside the GettextTranslator we now use
bindtextdomain as hints and store the locale directory which is finally
used to instantiate the translation catalog. For compatibility, both
methods are still calling their equivalents from thet gettext module.

Signed-off-by: Zygmunt Krynicki <email address hidden>

3497. By Zygmunt Krynicki

plainbox:clitools: add support for early logging

This patch adds support for early logging. Early logging can be enabled
even before command line argument parsing is attempted. It is perfect
for debugging command line handling code itself.

The feature is enabled by setting environment variables. There are three
new variables: PLAINBOX_DEBUG, PLAINBOX_TRACE and PLAINBOX_LOG_LEVEL.
The details of their behavior are documented in the plainbox manual
page.

Signed-off-by: Zygmunt Krynicki <email address hidden>

3496. By Zygmunt Krynicki

plainbox:provider_manager: fail gracefully in InstallCommand.get_command_epilog()

Signed-off-by: Zygmunt Krynicki <email address hidden>

3495. By Zygmunt Krynicki

plainbox:provider_manager: fix PEP-8 issue

Signed-off-by: Zygmunt Krynicki <email address hidden>

3494. By Zygmunt Krynicki

plainbox:provider_manager: add support for i18n in manage_py_extension

Signed-off-by: Zygmunt Krynicki <email address hidden>

3493. By Zygmunt Krynicki

plainbox:units:testplan: fix get_effective_category

This patch fixes TestPlan.get_effective_category() when no overrides
are being applied and the natural category is supposed to be returned.

Signed-off-by: Zygmunt Krynicki <email address hidden>

3492. By Zygmunt Krynicki

plainbox:commands: add 'session list --only-ids'

This patch adds a new argument to 'plainbox session list'. If invoked
with the --only-ids argument, the output is frendlier for scripting and
consists of session identifiers which are printed one per line.

Signed-off-by: Zygmunt Krynicki <email address hidden>

3491. By Zygmunt Krynicki

plainbox:session: fix typo in comment

Signed-off-by: Zygmunt Krynicki <email address hidden>

3490. By Zygmunt Krynicki

plainbox:session: fix FLAG_REWRITE_LOG_PATHNAMES_F

This patch fixes a embarrasing bug that gets triggered when session
resume is invoked with the rewrite-log-pathnames flag. The bug is a
result of later refactoring, which added binary flags in place of string
flags and set membership tests.

Signed-off-by: Zygmunt Krynicki <email address hidden>

3489. By Zygmunt Krynicki

plainbox:session: fix FLAG_IGNORE_JOB_CHECKSUMS_S

This patch fixes a embarrasing bug that gets triggered when session
resume is invoked with the ignore-job-checksums flag. The bug is a
result of later refactoring, which added binary flags in place of string
flags and set membership tests.

Signed-off-by: Zygmunt Krynicki <email address hidden>

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches