Project

General

Profile

Bug #8100

Asset precompilation split between foreman_docker & foreman_docker_engine

Added by Dominic Cleal over 5 years ago. Updated over 1 year ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
-
Target version:
Difficulty:
Triaged:
Bugzilla link:
Fixed in Releases:
Found in Releases:

Description

Asset precompilation is split over two directories at the moment as the engine name isn't redefined to "foreman_docker".

16:31 $ rake plugin:assets:precompile[foreman_docker]                                                                      16:31 $ ll ../foreman-docker/public/assets/*
../foreman-docker/public/assets/foreman_docker:
total 32
-rw-rw-r--. 1 dcleal dcleal  867 Oct 24 14:39 image_step-07895f03f97d6ccc96764ec538d5a120.js
-rw-rw-r--. 1 dcleal dcleal  366 Oct 24 14:39 image_step-07895f03f97d6ccc96764ec538d5a120.js.gz
-rw-rw-r--. 1 dcleal dcleal  867 Oct 24 14:39 image_step.js
-rw-rw-r--. 1 dcleal dcleal  366 Oct 24 14:39 image_step.js.gz
-rw-rw-r--. 1 dcleal dcleal 3159 Oct 24 14:39 terminal-22dce90f9325ffeae30af2c7b47e183d.css
-rw-rw-r--. 1 dcleal dcleal  567 Oct 24 14:39 terminal-22dce90f9325ffeae30af2c7b47e183d.css.gz
-rw-rw-r--. 1 dcleal dcleal 3159 Oct 24 14:39 terminal.css
-rw-rw-r--. 1 dcleal dcleal  567 Oct 24 14:39 terminal.css.gz

../foreman-docker/public/assets/foreman_docker_engine:
total 4
-rw-rw-r--. 1 dcleal dcleal 380 Oct 24 16:31 manifest.yml

Associated revisions

Revision 916c5290 (diff)
Added by Dominic Cleal over 5 years ago

fixes #8100 - make engine and asset precompilation name consistent

Ensures that when precompiling, both the assets and the manifest end up under
'foreman_docker' rather than one ending up under 'foreman_docker_engine'. Not
much functional effect except it keeps everything in one dir.

Revision 916c5290 (diff)
Added by Dominic Cleal over 5 years ago

fixes #8100 - make engine and asset precompilation name consistent

Ensures that when precompiling, both the assets and the manifest end up under
'foreman_docker' rather than one ending up under 'foreman_docker_engine'. Not
much functional effect except it keeps everything in one dir.

History

#1 Updated by The Foreman Bot over 5 years ago

  • Status changed from Assigned to Ready For Testing
  • Pull request https://github.com/theforeman/foreman-docker/pull/33 added
  • Pull request deleted ()

#2 Updated by Dominic Cleal over 5 years ago

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

Also available in: Atom PDF