Bug 1314563

Description of problem:

The 'foreman-rake katello:reindex' is often use to ensure that the Satellite (katello) database is kept in sync with backend services (e.g. pulp, candlepin).  While this is not something that is used all of the time, there are times that it is run as part of debugging and resolving issues.

It appears that the current re-index does not include Docker content (i.e. Katello::DockerManifest & Katello::DockerTag).  We should update this to be consistent.

Version-Release number of selected component (if applicable):
Satellite 6.2 snap 2

How reproducible:
always

Steps to Reproduce:

first, let's destroy the docker content in the katello database, leaving in place the data from backend services.  (Note: this should NOT be done on a production system, this is just for testing purposes)
1. foreman-rake console
2. User.current=User.first
3. Katello::DockerManifest.all.count   # keep the count
4. Katello::DockerTag.all.count        # keep the count

5. Katello::DockerManifest.destroy_all

6. Katello::DockerManifest.all.count   # should return 0
7. Katello::DockerTag.all.count        # should return 0

from outside of the console:
8. foreman-rake katello:reindex

from inside of the console:
9. Katello::DockerManifest.all.count   # should return same number as 3 above
10. Katello::DockerTag.all.count       # should return same number as 4 above

Actual results:

The counts from 3 & 9 and 4 & 10 do not match.

Expected results:

The counts should be the same, indicating that the docker content was re-indexed.  In addition, if look closer at the responses (without the '.count') the records should be similar (i.e. ids may be different, but the core content is the same).

Additional info: