Skip to content

Commit

Permalink
add big warning for extras.d usage
Browse files Browse the repository at this point in the history
This adds a warning for extras.d usage. This will give us something to
keep an eye on in logstash to build up the list of projects that will
break at Mitaka-1.

This also makes the deprecated handling done through a function, which
will hopefully make it more consistent in the future.

Change-Id: Icd393bc3e7095fe58be0fd13ef74fece3aa5c5f1
  • Loading branch information
sdague committed Oct 7, 2015
1 parent 1e7be9f commit 1de9e33
Showing 1 changed file with 16 additions and 0 deletions.
16 changes: 16 additions & 0 deletions functions-common
Expand Up @@ -179,6 +179,12 @@ function die_if_not_set {
$xtrace
}

function deprecated {
local text=$1
DEPRECATED_TEXT+="\n$text"
echo "WARNING: $text"
}

# Prints line number and "message" in error format
# err $LINENO "message"
function err {
Expand Down Expand Up @@ -1728,6 +1734,16 @@ function run_phase {
if [[ -d $TOP_DIR/extras.d ]]; then
for i in $TOP_DIR/extras.d/*.sh; do
[[ -r $i ]] && source $i $mode $phase
# NOTE(sdague): generate a big warning about using
# extras.d in an unsupported way which will let us track
# unsupported usage in the gate.
local exceptions="50-ironic.sh 60-ceph.sh 80-tempest.sh"
local extra=$(basename $i)
if [[ ! ( $exceptions =~ "$extra" ) ]]; then
deprecated "extras.d support is being removed in Mitaka-1"
deprecated "jobs for project $extra will break after that point"
deprecated "please move project to a supported devstack plugin model"
fi
done
fi
# the source phase corresponds to settings loading in plugins
Expand Down

0 comments on commit 1de9e33

Please sign in to comment.