October 2019 - Open Build Service Commits

[obs-commits] [openSUSE/open-build-service] faaae4: Call explicitly name on a class
by Henne Vogelsang 07 Oct '19

07 Oct '19

Branch: refs/heads/master Home: https://github.com/openSUSE/open-build-service Commit: faaae4aa6f71c55124f0792f081f591f959afd1d https://github.com/openSUSE/open-build-service/commit/faaae4aa6f71c55124f07… Author: Eduardo Navarro <enavarro(a)suse.com> Date: 2019-10-02 (Wed, 02 Oct 2019) Changed paths: M src/api/app/views/webui/comment/_comment_field.html.haml Log Message: ----------- Call explicitly name on a class This way we make sure that a string is returned, instead of a class. This should fix #8232. Co-authored-by: David Kang <dkang(a)suse.com> Commit: 627b069ff57bbdd5d115607db8b1cd4de731b73d https://github.com/openSUSE/open-build-service/commit/627b069ff57bbdd5d1156… Author: Henne Vogelsang <hvogel(a)opensuse.org> Date: 2019-10-07 (Mon, 07 Oct 2019) Changed paths: M src/api/app/views/webui/comment/_comment_field.html.haml Log Message: ----------- Merge pull request #8499 from eduardoj/fix_8232_add_name_to_class Call explicitly name on a class Compare: https://github.com/openSUSE/open-build-service/compare/562671e45f29...627b0…

1 0

[obs-commits] [openSUSE/open-build-service] d6c7d4: Prevent crashing on empty body requests
by Henne Vogelsang 07 Oct '19

07 Oct '19

Branch: refs/heads/master Home: https://github.com/openSUSE/open-build-service Commit: d6c7d4cb8891c6d67054874db62c2da1d8521abf https://github.com/openSUSE/open-build-service/commit/d6c7d4cb8891c6d670548… Author: Eduardo Navarro <enavarro(a)suse.com> Date: 2019-10-01 (Tue, 01 Oct 2019) Changed paths: M src/api/app/controllers/staging/staged_requests_controller.rb Log Message: ----------- Prevent crashing on empty body requests Also error messages are thrown in case of a XML request with: - empty body - invalid XML - no parseable requests' numbers Fixes #7349. Co-authored-by: David Kang <dkang(a)suse.com> Commit: 562671e45f29a6af373633046d1f709f50d9004e https://github.com/openSUSE/open-build-service/commit/562671e45f29a6af37363… Author: Henne Vogelsang <hvogel(a)opensuse.org> Date: 2019-10-07 (Mon, 07 Oct 2019) Changed paths: M src/api/app/controllers/staging/staged_requests_controller.rb Log Message: ----------- Merge pull request #8492 from eduardoj/fix_7349_dont_crash_on_empty_body Prevent crashing on empty body requests Compare: https://github.com/openSUSE/open-build-service/compare/839b9b9b6033...56267…

1 0

[obs-commits] [openSUSE/open-build-service] 80d2b8: Revert "Update sassc to version 2.2.1"
by Victor Pereira 07 Oct '19

07 Oct '19

Branch: refs/heads/master Home: https://github.com/openSUSE/open-build-service Commit: 80d2b8f2938fe46c6c11e2d344a4e5439229a4d2 https://github.com/openSUSE/open-build-service/commit/80d2b8f2938fe46c6c11e… Author: Saray Cabrera Padrón <scabrerapadron(a)suse.de> Date: 2019-10-07 (Mon, 07 Oct 2019) Changed paths: M src/api/Gemfile M src/api/Gemfile.lock Log Message: ----------- Revert "Update sassc to version 2.2.1" This update is causing problems when setting docker from scratch. This reverts commit edb07fad374f746ed75d6146ff8bf735288e8ee1. Commit: 839b9b9b6033036bbd2e7f555ded1b95c5acfd73 https://github.com/openSUSE/open-build-service/commit/839b9b9b6033036bbd2e7… Author: Victor Pereira <vpereira(a)suse.de> Date: 2019-10-07 (Mon, 07 Oct 2019) Changed paths: M src/api/Gemfile M src/api/Gemfile.lock Log Message: ----------- Merge pull request #8526 from saraycp/revert-sassc Revert "Update sassc to version 2.2.1" Compare: https://github.com/openSUSE/open-build-service/compare/f7976d305076...839b9…

1 0

[obs-commits] [openSUSE/open-build-service] b87f2d: Remove Y axis label width in monitor page plot gra...
by Victor Pereira 07 Oct '19

07 Oct '19

Branch: refs/heads/master Home: https://github.com/openSUSE/open-build-service Commit: b87f2d02e76812b127bbbbb9bf1bfb37d241afa6 https://github.com/openSUSE/open-build-service/commit/b87f2d02e76812b127bbb… Author: Eduardo Navarro <enavarro(a)suse.com> Date: 2019-10-04 (Fri, 04 Oct 2019) Changed paths: M src/api/app/assets/javascripts/webui/monitor.js Log Message: ----------- Remove Y axis label width in monitor page plot graphics Not setting a width in Y axis labels results in not breaking numbers with more than 4 digits in two lines. Fixes #8222. Commit: f7976d30507674e7ded336d7277f7d3b7220d791 https://github.com/openSUSE/open-build-service/commit/f7976d30507674e7ded33… Author: Victor Pereira <vpereira(a)suse.de> Date: 2019-10-07 (Mon, 07 Oct 2019) Changed paths: M src/api/app/assets/javascripts/webui/monitor.js Log Message: ----------- Merge pull request #8514 from eduardoj/fix_8222_remove_label_width Remove Y axis label width in monitor page plot graphics Compare: https://github.com/openSUSE/open-build-service/compare/514ea6d13003...f7976…

1 0

[obs-commits] [openSUSE/open-build-service] 62c302: Switch back to upstream influxdb-rails
by Henne Vogelsang 07 Oct '19

07 Oct '19

Branch: refs/heads/master Home: https://github.com/openSUSE/open-build-service Commit: 62c3021d99afa01fceda7f0f3c148ea56bcdd662 https://github.com/openSUSE/open-build-service/commit/62c3021d99afa01fceda7… Author: Henne Vogelsang <hvogel(a)opensuse.org> Date: 2019-10-07 (Mon, 07 Oct 2019) Changed paths: M src/api/Gemfile M src/api/Gemfile.lock M src/api/app/controllers/application_controller.rb M src/api/app/controllers/webui/webui_controller.rb M src/api/config/initializers/influxdb_rails.rb Log Message: ----------- Switch back to upstream influxdb-rails The latest beta covers everything we need, let's use it. Commit: 514ea6d130030754c7fd27d8e7c4b7266eb475be https://github.com/openSUSE/open-build-service/commit/514ea6d130030754c7fd2… Author: Henne Vogelsang <hvogel(a)opensuse.org> Date: 2019-10-07 (Mon, 07 Oct 2019) Changed paths: M src/api/Gemfile M src/api/Gemfile.lock M src/api/app/controllers/application_controller.rb M src/api/app/controllers/webui/webui_controller.rb M src/api/config/initializers/influxdb_rails.rb Log Message: ----------- Merge pull request #8515 from hennevogel/feature/update-influx Switch back to upstream influxdb-rails Compare: https://github.com/openSUSE/open-build-service/compare/b97e8b827a02...514ea…

1 0

[obs-commits] [openSUSE/open-build-service] 559937: Link to the staging workflow for the 'Staging' tab...
by Victor Pereira 07 Oct '19

07 Oct '19

Branch: refs/heads/master Home: https://github.com/openSUSE/open-build-service Commit: 5599376b75c5340282a61f69da50b573f415a791 https://github.com/openSUSE/open-build-service/commit/5599376b75c5340282a61… Author: Dany Marcoux <danymarcoux(a)gmail.com> Date: 2019-10-02 (Wed, 02 Oct 2019) Changed paths: M src/api/app/views/webui/project/_tabs.html.haml Log Message: ----------- Link to the staging workflow for the 'Staging' tab in the context of a staging project Closes #7357 Commit: b97e8b827a02e90b48d94d553b4ee95ecc4a9898 https://github.com/openSUSE/open-build-service/commit/b97e8b827a02e90b48d94… Author: Victor Pereira <vpereira(a)suse.de> Date: 2019-10-07 (Mon, 07 Oct 2019) Changed paths: M src/api/app/views/webui/project/_tabs.html.haml Log Message: ----------- Merge pull request #8475 from dmarcoux/issue-7357 Link to the staging workflow for the 'Staging' tab in the context of a staging project Compare: https://github.com/openSUSE/open-build-service/compare/faec0c073715...b97e8…

1 0

[obs-commits] [openSUSE/open-build-service] 09a3e8: Don't loop over partial in staging project show hi...
by Victor Pereira 07 Oct '19

07 Oct '19

Branch: refs/heads/master Home: https://github.com/openSUSE/open-build-service Commit: 09a3e87bfa2d5d09a313c0c0df22792c87a1e568 https://github.com/openSUSE/open-build-service/commit/09a3e87bfa2d5d09a313c… Author: Lukas Krause <lkrause(a)suse.de> Date: 2019-10-01 (Tue, 01 Oct 2019) Changed paths: M src/api/app/views/webui/staging/projects/_history_elements.html.haml R src/api/app/views/webui/staging/projects/_staging_project_created_history.html.haml R src/api/app/views/webui/staging/projects/_unstaged_staged_request_history.html.haml Log Message: ----------- Don't loop over partial in staging project show history Looping over partials is very inefficient and causes longer rendering times. Moving the logic towards rendering a collection is a bit difficult and the partials are only used at one place. So just getting rid of the partials and moving the code to the history template comes without any disadvantages. Related to #7350 Commit: faec0c073715a207d3b532461e96542454d93d7e https://github.com/openSUSE/open-build-service/commit/faec0c073715a207d3b53… Author: Victor Pereira <vpereira(a)suse.de> Date: 2019-10-07 (Mon, 07 Oct 2019) Changed paths: M src/api/app/views/webui/staging/projects/_history_elements.html.haml R src/api/app/views/webui/staging/projects/_staging_project_created_history.html.haml R src/api/app/views/webui/staging/projects/_unstaged_staged_request_history.html.haml Log Message: ----------- Merge pull request #8494 from krauselukas/issue_7350 WIP: Improve rendering/loading time of staging project show Compare: https://github.com/openSUSE/open-build-service/compare/bd5071b6603d...faec0…

1 0

[obs-commits] [openSUSE/open-build-service] 9c2758: Change reviews for staged and unstaged requests
by Victor Pereira 07 Oct '19

07 Oct '19

Branch: refs/heads/master Home: https://github.com/openSUSE/open-build-service Commit: 9c2758500297e8395093a8b5743df0a661e04720 https://github.com/openSUSE/open-build-service/commit/9c2758500297e8395093a… Author: David Kang <dkang(a)suse.com> Date: 2019-10-04 (Fri, 04 Oct 2019) Changed paths: M src/api/app/models/staging/stage_requests.rb M src/api/spec/controllers/staging/staged_requests_controller_spec.rb Log Message: ----------- Change reviews for staged and unstaged requests When we stage a request, now we add a review `by_project` and accept the existent review `by_group`. The reverse is done when we unstage a request. Fix #7344 Co-authored-by: Eduardo Navarro <enavarro(a)suse.com> Commit: bd5071b6603dc261a18f7ddc1baaec38a39045a0 https://github.com/openSUSE/open-build-service/commit/bd5071b6603dc261a18f7… Author: Victor Pereira <vpereira(a)suse.de> Date: 2019-10-07 (Mon, 07 Oct 2019) Changed paths: M src/api/app/models/staging/stage_requests.rb M src/api/spec/controllers/staging/staged_requests_controller_spec.rb Log Message: ----------- Merge pull request #8507 from DavidKang/staging-workflow/fix-7344 Change reviews for staged and unstaged requests Compare: https://github.com/openSUSE/open-build-service/compare/9be761719d7f...bd507…

1 0

[obs-commits] [openSUSE/open-build-service] 724a0f: Make prjconf a nested resource inside projects
by Victor Pereira 07 Oct '19

07 Oct '19

Branch: refs/heads/master Home: https://github.com/openSUSE/open-build-service Commit: 724a0f123340f72e4e97634f9886dca06f962698 https://github.com/openSUSE/open-build-service/commit/724a0f123340f72e4e976… Author: Victor Pereira <vpereira(a)suse.de> Date: 2019-10-02 (Wed, 02 Oct 2019) Changed paths: M src/api/app/views/webui/project/prjconf.html.haml M src/api/app/views/webui/projects/project_configuration/show.html.haml M src/api/config/routes/webui_routes.rb M src/api/spec/controllers/webui/projects/project_configuration_controller_spec.rb Log Message: ----------- Make prjconf a nested resource inside projects To simplify the routes, move the project/prjconf/:project routes to the `resources :projects` Commit: 9e85c4e1c627ca69e29aaaf4a6925127cc8492a4 https://github.com/openSUSE/open-build-service/commit/9e85c4e1c627ca69e29aa… Author: Victor Pereira <vpereira(a)suse.de> Date: 2019-10-02 (Wed, 02 Oct 2019) Changed paths: M src/api/config/routes/webui_routes.rb Log Message: ----------- Add redirect to the old prjconf Commit: 9be761719d7f54dcc1beb6b10af5a261910fa485 https://github.com/openSUSE/open-build-service/commit/9be761719d7f54dcc1beb… Author: Victor Pereira <vpereira(a)suse.de> Date: 2019-10-07 (Mon, 07 Oct 2019) Changed paths: M src/api/app/views/webui/project/prjconf.html.haml M src/api/app/views/webui/projects/project_configuration/show.html.haml M src/api/config/routes/webui_routes.rb M src/api/spec/controllers/webui/projects/project_configuration_controller_spec.rb Log Message: ----------- Merge pull request #8496 from vpereira/move_prjconf_routes_to_project Make prjconf routes resourceful Compare: https://github.com/openSUSE/open-build-service/compare/a2bf72eb5c32...9be76…

1 0

[obs-commits] [openSUSE/open-build-service] bb6d72: Update uniform_notifier to version 1.13.0
by depfu[bot] 07 Oct '19

07 Oct '19

Branch: refs/heads/depfu/update/srcapi/uniform_notifier-1.13.0 Home: https://github.com/openSUSE/open-build-service Commit: bb6d72679ec77ad9ffa6e3df79539f69b6c2efb0 https://github.com/openSUSE/open-build-service/commit/bb6d72679ec77ad9ffa6e… Author: depfu[bot] <23717796+depfu[bot](a)users.noreply.github.com> Date: 2019-10-07 (Mon, 07 Oct 2019) Changed paths: M src/api/Gemfile.lock Log Message: ----------- Update uniform_notifier to version 1.13.0

1 0