Project

General

Profile

Actions

Refactor #20024

closed

Improve performance of rhsm fact importer

Added by Shimon Shtein almost 7 years ago. Updated over 5 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Hosts
Target version:
Branch:
Difficulty:
Triaged:
Fixed in Releases:
Found in Releases:

Description

Today it takes ~600ms for each host, even when the facts are the same.
This is caused by wrong handling of hierarchy - the non-leaf levels are deleted and then re-added on each facts upload.


Related issues 1 (0 open1 closed)

Related to foreman-tasks - Bug #19951: updating host can cause large values to get passed via dynflow envelope, slowing task processingRejected06/07/2017Actions
Actions #1

Updated by Shimon Shtein almost 7 years ago

  • Related to Bug #19951: updating host can cause large values to get passed via dynflow envelope, slowing task processing added
Actions #2

Updated by The Foreman Bot almost 7 years ago

  • Status changed from New to Ready For Testing
  • Pull request https://github.com/Katello/katello/pull/6825 added
Actions #3

Updated by Shimon Shtein almost 7 years ago

  • Bugzilla link set to 1458857
Actions #4

Updated by Adam Ruzicka almost 7 years ago

  • Target version set to 1.14.1
Actions #5

Updated by Shimon Shtein almost 7 years ago

  • Status changed from Ready For Testing to Closed
  • % Done changed from 0 to 100
Actions #6

Updated by Daniel Lobato Garcia almost 7 years ago

  • translation missing: en.field_release set to 250

This would be nice for 3.4.2, but not without tests :/

Actions #7

Updated by Shimon Shtein almost 7 years ago

There is a basic importer test that tests basic fact importing: https://github.com/Katello/katello/blob/master/test/models/rhsm_fact_importer_test.rb

Actions #8

Updated by Eric Helms almost 7 years ago

  • translation missing: en.field_release deleted (250)

Please try not to set release outside of our weekly triage process. Unsetting the release so it can be triaged.

Actions #9

Updated by Justin Sherrill almost 7 years ago

  • Category set to Hosts
  • translation missing: en.field_release set to 258
Actions #10

Updated by Eric Helms almost 7 years ago

  • translation missing: en.field_release changed from 258 to 267
Actions #11

Updated by Eric Helms almost 7 years ago

  • translation missing: en.field_release changed from 267 to 258
Actions

Also available in: Atom PDF