kparal (Kamil Páral)Administrator
User

Today

  • Clear sailing ahead.

Tomorrow

  • Clear sailing ahead.

Tuesday

  • Clear sailing ahead.

User Details

User Since
Nov 12 2013, 2:18 PM (192 w, 4 d)
Roles
Administrator
Availability
Available

Recent Activity

Mon, Jul 17

kparal edited the content of 20170717-fedoraqadevel.
Mon, Jul 17, 8:32 AM

Fri, Jul 14

kparal created D1221: remove dnf cache support.
Fri, Jul 14, 12:41 PM

Thu, Jul 13

kparal closed T13: Create list of situations in which depcheck should pass and fail, a subtask of T12: Create comprehensive functional tests for package level checks, as "Wontfix".
Thu, Jul 13, 1:40 PM · package-checks
kparal closed T13: Create list of situations in which depcheck should pass and fail as "Wontfix".

Depcheck is now replaced by #task-rpmdeplint .

Thu, Jul 13, 1:40 PM · task-depcheck
kparal closed T384: depcheck misses dropped subpackages as "Wontfix".

Depcheck is now replaced by #task-rpmdeplint .

Thu, Jul 13, 1:40 PM · task-depcheck
kparal closed T289: Investigate ways to get repo information quicker as "Wontfix".

Depcheck is now replaced by #task-rpmdeplint .

Thu, Jul 13, 1:39 PM · task-depcheck
kparal added a project to T357: depcheck: detect cases where newer build of package X breaks some other existing package Y: Restricted Project.
Thu, Jul 13, 1:39 PM · Restricted Project, task-depcheck
kparal added a project to T562: Support new RPM features in depcheck: Restricted Project.
Thu, Jul 13, 1:38 PM · Restricted Project, task-depcheck
kparal closed T366: depcheck is failing with strange "none of the providers can be installed" for a recent kernel update as "Wontfix".

Depcheck is now replaced by #task-rpmdeplint .

Thu, Jul 13, 1:37 PM · task-depcheck
kparal closed T372: depcheck is failing on some multilib builds when they install fine on a regular system as "Wontfix".

Depcheck is now replaced by #task-rpmdeplint .

Thu, Jul 13, 1:37 PM · task-depcheck
kparal closed T878: replace depcheck with rpmdeplint as "Resolved".

Depcheck is now officially dead! Whooo!

Thu, Jul 13, 1:33 PM · Restricted Project, task-depcheck, Restricted Project, new-check-ideas
kparal closed T962: F26 is released as "Resolved".

@mkrizek deployed all to production.

Thu, Jul 13, 11:50 AM · infrastructure

Wed, Jul 12

kparal created D1220: rpm_utils: remove is_installed().
Wed, Jul 12, 2:01 PM
kparal added a comment to D1218: add SKIPPED outcome.

For example in Bodhi right now I think the note does *not* appear anywhere... so that might be something we should add?

Wed, Jul 12, 10:43 AM
kparal added a comment to T962: F26 is released.

Josef, can you please deploy the change from D1219 onto taskotron-dev and verify that the config is adjusted properly? Thanks a lot.

Wed, Jul 12, 10:22 AM · infrastructure
kparal closed D1219: taskotron-client: replace vars in yumrepoinfo.conf.

Pushed to ansible as 3952106f493.

Wed, Jul 12, 10:21 AM · infrastructure

Tue, Jul 11

kparal created D1219: taskotron-client: replace vars in yumrepoinfo.conf.
Tue, Jul 11, 11:27 AM · infrastructure
kparal created T962: F26 is released.
Tue, Jul 11, 8:12 AM · infrastructure
kparal added a comment to D1218: add SKIPPED outcome.

Ohh no, skipping the test suite/linter was accidental... due to me not understand Phabricator UI I guess. :-) I didn't see any button to skip them or run them so not sure what I did wrong.

Tue, Jul 11, 5:50 AM

Mon, Jul 10

kparal added a comment to D1218: add SKIPPED outcome.

Also, any reason for skipping the test suite and lint when creating this diff? I don't see any unit test failures, but some lint warnings are present.

Mon, Jul 10, 11:51 AM
kparal added a comment to D1218: add SKIPPED outcome.

Out of curiosity, why not use PASSED for them? I'm not saying SKIPPED is a bad idea (I encountered this myself lately and would use it if it was present), but in my case the two outcomes had the same meaning in the end (in terms of gating, both skipped and passed means thumbs up). What is your scenario where you need to distinguish the two?

Mon, Jul 10, 11:47 AM

Fri, Jun 30

kparal triaged T960: publish ResultsDB package to PyPI as "Wishlist" priority.
Fri, Jun 30, 1:23 PM · resultsdb
kparal added a comment to T957: libtaskotron: unicode issues and dnf --disableplugin=noroot broken on F26.

This has been partially "fixed" in D1216 - figuring out whether packages are installed now works under a non-root account, as long as you use plain package names/file paths as requirements. That should cover the major issues until we replace this whole code.

Fri, Jun 30, 1:19 PM · Restricted Project, libtaskotron
kparal created D1217: requirements: limit Flask-RESTful to <= 0.3.5.
Fri, Jun 30, 1:00 PM
kparal added a revision to T961: Flask-RESTful==0.3.6 breaks unit tests: D1217: requirements: limit Flask-RESTful to <= 0.3.5.
Fri, Jun 30, 1:00 PM · resultsdb
kparal added a comment to T961: Flask-RESTful==0.3.6 breaks unit tests.

You can see the error log in F99608.

Fri, Jun 30, 12:56 PM · resultsdb
kparal renamed T961: Flask-RESTful==0.3.6 breaks unit tests from "Tests failing with the environment in requirements.txt" to "Flask-RESTful==0.3.6 breaks unit tests".
Fri, Jun 30, 12:53 PM · resultsdb
kparal triaged T961: Flask-RESTful==0.3.6 breaks unit tests as "High" priority.

The problem is in Flask-RESTful==0.3.6. If you use <= 0.3.5, everything works for me. I'll prepare a patch to restrict the max version number. And somebody should figure out why that happens and whether it's something that we need to fix in our tests, or whether it's a flask-restful bug.

Fri, Jun 30, 12:53 PM · resultsdb
kparal created D1216: rpm_utils: only support packages in is_installed().
Fri, Jun 30, 11:49 AM
kparal added a revision to T957: libtaskotron: unicode issues and dnf --disableplugin=noroot broken on F26: D1216: rpm_utils: only support packages in is_installed().
Fri, Jun 30, 11:49 AM · Restricted Project, libtaskotron

Thu, Jun 29

kparal added a comment to T878: replace depcheck with rpmdeplint.

I have obsoleted depcheck project on bitbucket and disabled depcheck project and mirror on Phab. @mkrizek removed task-depcheck from taskotron-dev and taskotron-stg. The only thing missing is removing it also from production, once F26 Final freeze is over.

Thu, Jun 29, 12:46 PM · Restricted Project, task-depcheck, Restricted Project, new-check-ideas
kparal archived task-depcheck.
Thu, Jun 29, 12:15 PM
kparal removed a member for task-depcheck: kparal.
Thu, Jun 29, 12:15 PM
kparal removed a watcher for task-depcheck: kparal.
Thu, Jun 29, 12:15 PM
kparal edited Description on task-depcheck.
Thu, Jun 29, 12:13 PM
kparal edited Description on repos-taskotron.
Thu, Jun 29, 12:10 PM
kparal created D1215: namespaces.yaml: synchronize with current deployment.
Thu, Jun 29, 11:04 AM

Tue, Jun 27

kparal created D1214: resultsdb_directive: log submitted results.
Tue, Jun 27, 2:45 PM

Sun, Jun 25

kparal added a comment to T878: replace depcheck with rpmdeplint.

I have a blog post draft ready that I can send to fedora planet and test-announce and devel-announce once taskotron production is working again.

Sun, Jun 25, 5:05 PM · Restricted Project, task-depcheck, Restricted Project, new-check-ideas

Jun 21 2017

kparal added a comment to T878: replace depcheck with rpmdeplint.

Both FMN and Bodhi new releases have been deployed. I'll write a blogpost announcing depcheck -> rpmdeplint switch, and then we can disable depcheck, I believe.

Jun 21 2017, 6:11 PM · Restricted Project, task-depcheck, Restricted Project, new-check-ideas

Jun 20 2017

kparal requested changes to D1196: *WIP* More hacky support, but for containers.

Some of the test suite failures seem to be caused by missing dependencies (import hawkey or import libvirt fails). Follow the readme to install dependencies. The test suite needs to pass so that we can continue refactoring while being sure we haven't broken anything.

Jun 20 2017, 3:33 PM · libtaskotron
kparal added a comment to D1196: *WIP* More hacky support, but for containers.

Also please make sure you're running arc diff feature/ansiblize when updating this.

Jun 20 2017, 11:21 AM · libtaskotron
kparal added a comment to D1196: *WIP* More hacky support, but for containers.

That's weird, it's still showing more than just your changes in the diff. I merged develop into feature/ansiblize. Can you try to rebase again? I'd like to read the clean diff once more, and in the current state it's not clear which of the changes are yours.

Jun 20 2017, 9:54 AM · libtaskotron

Jun 19 2017

kparal closed T954: allow resultsdb directive to use the root 'task_output' dir as a log link as "Resolved".

Solved, default changed, closing.

Jun 19 2017, 12:32 PM · Restricted Project, libtaskotron
kparal moved T957: libtaskotron: unicode issues and dnf --disableplugin=noroot broken on F26 from Restricted Project Column to Restricted Project Column on the Restricted Project board.
Jun 19 2017, 11:55 AM · Restricted Project, libtaskotron
kparal added a subtask for T957: libtaskotron: unicode issues and dnf --disableplugin=noroot broken on F26: T939: Support New Test Invocation Standard.
Jun 19 2017, 11:53 AM · Restricted Project, libtaskotron
kparal added a parent task for T939: Support New Test Invocation Standard: T957: libtaskotron: unicode issues and dnf --disableplugin=noroot broken on F26.
Jun 19 2017, 11:53 AM · libtaskotron
kparal added a comment to T957: libtaskotron: unicode issues and dnf --disableplugin=noroot broken on F26.

So, I looked into the brave new world in T939. As it seems, we'll likely remove sudo support, since it requires to run tasks as root. So that's first part "resolved". The current WIP code in D1195 also removes all code that uses dnf install and uses ansible for that instead. If that turns out of be the way we handle things, that's the second half of this "resolved", we'll see.

Jun 19 2017, 11:53 AM · Restricted Project, libtaskotron
kparal added a comment to T878: replace depcheck with rpmdeplint.

rpmdeplint has been deployed to production. I've compared depcheck and rpmdeplint results between 2017-06-16T08:00 and 2017-06-19T08:00 and there are no important differences. Now waiting for new FMN and Bodhi release.

Jun 19 2017, 10:52 AM · Restricted Project, task-depcheck, Restricted Project, new-check-ideas
kparal updated subscribers of D1210: resultsdb_directive: change default log link to artifactsdir.

As an example, say rpmdeplint doesn't set custom artifact in resultyaml (which it does). The default link would go from:
https://taskotron.fedoraproject.org/artifacts/all/43567c16-54c3-11e7-b089-5254008e42f6/task_output/taskotron.log.gz
to:
https://taskotron.fedoraproject.org/artifacts/all/43567c16-54c3-11e7-b089-5254008e42f6/
(Please note that taskotron logs should be in that root dir, instead of being in task_output, it's a bug).

Jun 19 2017, 8:26 AM

Jun 15 2017

kparal added a comment to T954: allow resultsdb directive to use the root 'task_output' dir as a log link.

check_modulemd fix has been pushed and is working fine. However, after some thinking, changing the default to work exactly the same way makes a lot of sense, I believe. Please review D1210.

Jun 15 2017, 4:08 PM · Restricted Project, libtaskotron
kparal created D1210: resultsdb_directive: change default log link to artifactsdir.
Jun 15 2017, 4:06 PM
kparal added a revision to T954: allow resultsdb directive to use the root 'task_output' dir as a log link: D1210: resultsdb_directive: change default log link to artifactsdir.
Jun 15 2017, 4:06 PM · Restricted Project, libtaskotron
kparal reassigned T949: FMN: Add/Modify filters so that users can get all taskotron results for a certain item from kparal to tflink.

Reassigning back to @tflink for clarification.

Jun 15 2017, 2:10 PM · infrastructure

Jun 14 2017

kparal moved T954: allow resultsdb directive to use the root 'task_output' dir as a log link from Restricted Project Column to Restricted Project Column on the Restricted Project board.
Jun 14 2017, 1:53 PM · Restricted Project, libtaskotron
kparal added a project to T954: allow resultsdb directive to use the root 'task_output' dir as a log link: Restricted Project.

This could do it: https://github.com/fedora-modularity/check_modulemd/pull/22

Jun 14 2017, 1:52 PM · Restricted Project, libtaskotron
kparal claimed T954: allow resultsdb directive to use the root 'task_output' dir as a log link.
Jun 14 2017, 12:26 PM · Restricted Project, libtaskotron
kparal added a comment to T958: migrate docs to readthedocs.

Koji request here: https://pagure.io/koji/issue/458

Jun 14 2017, 10:25 AM · libtaskotron
kparal added a comment to T958: migrate docs to readthedocs.

Is it only Koji that we're missing on pypi, or more? Because when I look at koji rpm, it doesn't include any C modules. It should be easy to submit it to Koji (or ask the maintainers to do it).

Jun 14 2017, 10:12 AM · libtaskotron
kparal added a comment to T878: replace depcheck with rpmdeplint.

Bodhi pull request pushed and Tasks/dist.rpmdeplint created. We're now waiting for Beta freeze to end so that @mkrizek can push rpmdeplint to production.

Jun 14 2017, 7:50 AM · Restricted Project, task-depcheck, Restricted Project, new-check-ideas
kparal created T959: release switcher gets cropped if too many releases are enabled.
Jun 14 2017, 7:46 AM · blockerbugs

Jun 13 2017

kparal created T957: libtaskotron: unicode issues and dnf --disableplugin=noroot broken on F26.
Jun 13 2017, 1:24 PM · Restricted Project, libtaskotron
kparal added a comment to D1196: *WIP* More hacky support, but for containers.

Martin just landed his commit in a feature branch, can you please rebase against that?

Jun 13 2017, 11:59 AM · libtaskotron
kparal updated subscribers of D1195: Support for Ansible Tasks.
Jun 13 2017, 11:10 AM
kparal added a comment to D1195: Support for Ansible Tasks.

There's no point further waiting here. Please merge with latest develop and push to a feature branch, thanks. It will also help @roshi push D1196.

Jun 13 2017, 11:05 AM

Jun 6 2017

kparal reopened T881: all gzipped artifacts are served as text/plain as "Open".

I think we need to solve this for all file types (not just whitelist specific ones), or disable the compression (and use a compressed filesystem or something, if needed). Images still seem broken:
https://taskotron-dev.fedoraproject.org/artifacts/all/fd064e20-47a8-11e7-bdc7-525400cb0b45/task_output/avocado-result/html/images/sort_asc.png.gz
They are served with Content-Type: text/css.

Jun 6 2017, 2:23 PM · infrastructure

Jun 5 2017

kparal closed T956: testcases wildcarded query crashes as "Invalid".

Doh, again I forgot the difference between API endpoint and our frontend.
https://taskotron.fedoraproject.org/resultsdb_api//api/v2.0/testcases?name:like=dist.modulemd*

Jun 5 2017, 11:18 AM · resultsdb
kparal created T956: testcases wildcarded query crashes.
Jun 5 2017, 11:17 AM · resultsdb
kparal edited the content of 20170605-fedoraqadevel.
Jun 5 2017, 8:42 AM

Jun 2 2017

kparal added a comment to T881: all gzipped artifacts are served as text/plain.

Can we please disable gzipping everywhere until we have a better solution? Currently even the production reports are broken:
https://taskotron.fedoraproject.org/artifacts/all/dedb65bc-4793-11e7-a421-5254008e42f6/task_output/avocado-result/html/results.html
And people start using those.

Jun 2 2017, 2:01 PM · infrastructure

Jun 1 2017

kparal created T955: distgit_directive: allow missing files.
Jun 1 2017, 2:46 PM · libtaskotron
kparal added a comment to T878: replace depcheck with rpmdeplint.

FMN pull request pushed and will be part of a new FMN release after Beta freeze.

Jun 1 2017, 10:13 AM · Restricted Project, task-depcheck, Restricted Project, new-check-ideas

May 31 2017

kparal closed T953: support wildcards in FMN filters as "Resolved".

This has been pushed to FMN source code and should be deployed once Beta freeze is lifted.

May 31 2017, 7:04 AM · Restricted Project, infrastructure

May 30 2017

kparal added a comment to D1195: Support for Ansible Tasks.

@mkrizek, instead of creating a new repo for each converted task (https://pagure.io/task-rpmlint-ansible), would it be possible to instead create a new branch (like feature/ansible or ansible) in the existing repo? I feel like we have too many repos already :-)

May 30 2017, 2:39 PM

May 26 2017

kparal moved T953: support wildcards in FMN filters from Restricted Project Column to Restricted Project Column on the Restricted Project board.
May 26 2017, 4:25 PM · Restricted Project, infrastructure
kparal added a project to T953: support wildcards in FMN filters: Restricted Project.

Pull request here: https://github.com/fedora-infra/fmn/pull/197

May 26 2017, 4:24 PM · Restricted Project, infrastructure
kparal created T953: support wildcards in FMN filters.
May 26 2017, 2:19 PM · Restricted Project, infrastructure
kparal added a comment to T878: replace depcheck with rpmdeplint.

Here's a FMN pull request I intend to send:
https://github.com/kparal/fmn/compare/develop...kparal:rpmdeplint
Not only I renamed depcheck to rpmdeplint, but I also added abicheck as an important task to automatically notify people on failure. We don't get any false negatives reports, it's pretty important, and currently runs just on critical path packages. Abicheck authors have been asking for increased visibility for some time. Does it make sense, do you agree?

May 26 2017, 1:05 PM · Restricted Project, task-depcheck, Restricted Project, new-check-ideas
kparal added a comment to D1191: A stomp messaging plugin..

@tflink added you into taskotron group, please try again

May 26 2017, 1:04 PM
kparal edited reviewers for D1191: A stomp messaging plugin., added: resultsdb; removed: kparal.
May 26 2017, 10:47 AM

May 23 2017

kparal added a comment to T878: replace depcheck with rpmdeplint.

OK. I also removed type=bodhi_update reporting, because we want to get rid of it, and this change is the best chance to push it forward (in the worst case, I'll also prepare a bodhi patch). Now we're waiting for beta freeze to be lifted so that we can push this to production. Also, I'll look at necessary FMN changes and bodhi changes.

May 23 2017, 12:01 PM · Restricted Project, task-depcheck, Restricted Project, new-check-ideas
kparal claimed T878: replace depcheck with rpmdeplint.
May 23 2017, 11:47 AM · Restricted Project, task-depcheck, Restricted Project, new-check-ideas
kparal reopened T835: fedora-phabricator repo is not available for Fedora 25 as "Open".

Our documentation links to https://repos.fedorapeople.org/repos/tflink/phabricator/ . I can change that, but kanarip's packages don't work with our Phab instance:

$ arc list
Exception
[cURL/77] (https://phab.qa.fedoraproject.org/api/user.whoami) <CURLE_SSL_CACERT_BADFILE> The SSL CA Bundles that we tried to use could not be read or are not formatted correctly.
(Run with `--trace` for a full exception trace.)
$ arc list --trace
 ARGV  '/usr/share/arcanist/bin/../scripts/arcanist.php' 'list' '--trace'
 LOAD  Loaded "phutil" from "/usr/share/libphutil/src".
 LOAD  Loaded "arcanist" from "/usr/share/arcanist/src".
Config: Reading user configuration file "/home/kparal/.arcrc"...
Config: Did not find system configuration at "/etc/arcconfig".
Working Copy: Reading .arcconfig from "/home/kparal/devel/taskotron/libtaskotron/.arcconfig".
Working Copy: Path "/home/kparal/devel/taskotron/libtaskotron" is part of `git` working copy "/home/kparal/devel/taskotron/libtaskotron".
Working Copy: Project root is at "/home/kparal/devel/taskotron/libtaskotron".
Config: Did not find local configuration at "/home/kparal/devel/taskotron/libtaskotron/.git/arc/config".
>>> [0] <conduit> user.whoami() <bytes = 117>
>>> [1] <http> https://phab.qa.fedoraproject.org/api/user.whoami
<<< [1] <http> 297,727 us
<<< [0] <conduit> 297,936 us
May 23 2017, 9:40 AM · Restricted Project, infrastructure
kparal added inline comments to D1196: *WIP* More hacky support, but for containers.
May 23 2017, 8:51 AM · libtaskotron

May 22 2017

kparal added a comment to T878: replace depcheck with rpmdeplint.

So, here's my most awesome proposal - let's use this for now:

dist.rpmdeplint

Once we also start checking repoclosure with it, we can extend it to

dist.rpmdeplint
dist.rpmdeplint.sat
dist.rpmdeplint.repoclosure

Which means dist.rpmdeplint would be the overall outcome of .sat and .repoclosure.

May 22 2017, 6:06 PM · Restricted Project, task-depcheck, Restricted Project, new-check-ideas
kparal closed T894: a race condition between downloading i386 and x86_64 packages as "Resolved".

D1178 has been pushed and seems to be working well. This problem has been therefore solved for #task-rpmdeplint. Closing the ticket. The problem is still valid for task-depcheck, now we just need to finish T878 and get rid of depcheck.

May 22 2017, 2:47 PM · Restricted Project, task-depcheck, Restricted Project
kparal added a comment to D1178: run per-arch.

This is working well in develop, pushing to master as well.

May 22 2017, 2:42 PM
kparal added inline comments to D1196: *WIP* More hacky support, but for containers.
May 22 2017, 2:21 PM · libtaskotron
kparal added a comment to T939: Support New Test Invocation Standard.

I understand the motivation - being able to transfer the task between different test systems without any changes. What I don't understand is how localhost-only ansible execution is related here. When I execute a task through ansible locally, and when I execute it remotely from a different machine, doesn't it look the same to the task itself? Is there a different environment or some behavioral changes that the task needs to adapt to? If the execution looks the same to the task regardless of ansible mode, the end goal is achieved (the task doesn't need to be adjusted when moving between test systems), and the particular ansible execution mode then can be left to the test system to implement as it fits best for it.

May 22 2017, 1:49 PM · libtaskotron
kparal edited the content of 20170522-fedoraqadevel.
May 22 2017, 10:45 AM

May 19 2017

kparal added a comment to D1178: run per-arch.

@jskladan @mkrizek: I'd like to push this into develop branch soon, so that I can test how well it works in taskotron-dev. Please review, otherwise I'll do the push early next week. Thanks.

May 19 2017, 2:17 PM
kparal added inline comments to D1196: *WIP* More hacky support, but for containers.
May 19 2017, 2:03 PM · libtaskotron
kparal updated the diff for D1199: README: add testcloud dependency.
  • add libvirt-python to readme instead, require fixed testcloud from pypi
May 19 2017, 1:36 PM
kparal added a comment to D1199: README: add testcloud dependency.

OK, so, testcloud deps were fixed in D1200 and pushed to pypi.

May 19 2017, 1:34 PM
kparal created D1200: setup: specify package deps.
May 19 2017, 11:31 AM

May 18 2017

kparal planned changes to D1199: README: add testcloud dependency.

Ah, good catch. The actual problem was with missing libvirt-python. That's a dependency of testcloud, but it clearly doesn't state it on the pypi package. So, we should add the dep to the testcloud package (I'll create a patch). The drawback is that libvirt-python is a bit problematic to install from pypi, it requires to compile some libvirt modules, so it might be actually easier to install it as an rpm anyway. Will look into it.

May 18 2017, 2:08 PM
kparal created D1199: README: add testcloud dependency.
May 18 2017, 1:36 PM
kparal updated the diff for D1198: change documentation theme.
  • fix running sphinx-build from virtualenv and use better approach to add stylesheet (the old one fails with newer sphinx)
May 18 2017, 12:59 PM
kparal updated the test plan for D1198: change documentation theme.
May 18 2017, 11:28 AM
kparal created D1198: change documentation theme.
May 18 2017, 10:59 AM