From 2131e2aa5fedf4ebeecf1e719054061e950b4e92 Mon Sep 17 00:00:00 2001 From: Alexandru Avadanii Date: Fri, 13 May 2016 17:51:20 +0200 Subject: [PATCH] Rebase: p/opnfv-fuel: Drop upstream patches. While we're at it, fix minor context in remaining patch. --- ...py-stdout-not-consumed-when-deploying-cha.patch | 66 ---------------------- ...deploy.sh-do-not-expect-a-parameter-for-h.patch | 47 --------------- ...23-deploy.py-add-multiple-bridges-support.patch | 3 +- 3 files changed, 1 insertion(+), 115 deletions(-) delete mode 100644 patches/opnfv-fuel/0010-deployment.py-stdout-not-consumed-when-deploying-cha.patch delete mode 100644 patches/opnfv-fuel/0012-deploy.sh-do-not-expect-a-parameter-for-h.patch diff --git a/patches/opnfv-fuel/0010-deployment.py-stdout-not-consumed-when-deploying-cha.patch b/patches/opnfv-fuel/0010-deployment.py-stdout-not-consumed-when-deploying-cha.patch deleted file mode 100644 index 584413ec..00000000 --- a/patches/opnfv-fuel/0010-deployment.py-stdout-not-consumed-when-deploying-cha.patch +++ /dev/null @@ -1,66 +0,0 @@ -From: Josep Puigdemont -Date: Wed, 4 May 2016 14:27:23 +0200 -Subject: [PATCH] deployment.py: stdout not consumed when deploying changes - -During the automatic deployment, when the environment is ready to be -deployed, the deploy.py script will spawn a shell process that will -perform the command "fuel deploy-changes". The standard output of this -process is then piped to a "tee" process, which redirects the output -to the standard output of the shell process, and to a file named -cloud.log. The file is monitored by the deploy script to find out the -status of the deployment, and print it to the log file of the automatic -deployment script, including percentages for each node being -provisioned. However, the deploy script never consumes the standard -output of the shell process. If the shell process produces enough -output, its standard output buffer will fill up, thus making the tee -process block trying to write to its standard output, and the cloud.log -file will not be updated. At this point, the deploy process, which is -monitoring cloud.log, will not detect any progress in the deployment, -and eventually it will time out and assume the deployment failed, -although it might have finished fine after that. - -The solution here is to remove the "tee" process from the shell command, -and instead redirect standard output to the cloud.log file. -Another solution would be to actually parse the standard output of the -shell command from the deploy script itself, but that would require a -bit more work, as reading a line at a time might block the script. - -Finally, with this patch the cloud.log file won't be deleted unless the -shell process has already finished. - -Change-Id: I03a77be42d220b1606e48fc4ca35e22d73a6e583 -Signed-off-by: Josep Puigdemont ---- - deploy/cloud/deployment.py | 12 +++++++++--- - 1 file changed, 9 insertions(+), 3 deletions(-) - -diff --git a/deploy/cloud/deployment.py b/deploy/cloud/deployment.py -index 306abf0..0127d2a 100644 ---- a/deploy/cloud/deployment.py -+++ b/deploy/cloud/deployment.py -@@ -101,8 +101,8 @@ class Deployment(object): - LOG_FILE = 'cloud.log' - - log('Starting deployment of environment %s' % self.env_id) -- run_proc('fuel --env %s deploy-changes | strings | tee %s' -- % (self.env_id, LOG_FILE)) -+ p = run_proc('fuel --env %s deploy-changes | strings > %s' -+ % (self.env_id, LOG_FILE)) - - ready = False - for i in range(int(self.deploy_timeout)): -@@ -119,7 +119,13 @@ class Deployment(object): - break - else: - time.sleep(SLEEP_TIME) -- delete(LOG_FILE) -+ -+ p.poll() -+ if p.returncode == None: -+ log('The process deploying the changes has not yet finished.') -+ log('''The file %s won't be deleted''' % LOG_FILE) -+ else: -+ delete(LOG_FILE) - - if ready: - log('Environment %s successfully deployed' % self.env_id) diff --git a/patches/opnfv-fuel/0012-deploy.sh-do-not-expect-a-parameter-for-h.patch b/patches/opnfv-fuel/0012-deploy.sh-do-not-expect-a-parameter-for-h.patch deleted file mode 100644 index f515aab6..00000000 --- a/patches/opnfv-fuel/0012-deploy.sh-do-not-expect-a-parameter-for-h.patch +++ /dev/null @@ -1,47 +0,0 @@ -From: Josep Puigdemont -Date: Wed, 4 May 2016 14:27:23 +0200 -Subject: [PATCH] deploy.sh: do not expect a parameter for -h - -If -h was given as a parameter to the script, it would report an error -as it expected a parameter, and if it was called as the only parameter, -it would run deploy.py as if "old style" parameters had been given, thus -showing the usage for the python script, instead of the expected usage -message for this script. - -Update the usage message to include -h. - -Change-Id: I0930936962c1cb479ec4409ff114cd60a386b276 -Signed-off-by: Josep Puigdemont ---- - ci/deploy.sh | 4 +++- - 1 file changed, 3 insertions(+), 1 deletion(-) - -diff --git a/ci/deploy.sh b/ci/deploy.sh -index d83bba2..dc13f1c 100755 ---- a/ci/deploy.sh -+++ b/ci/deploy.sh -@@ -40,6 +40,7 @@ OPTIONS: - -f Deploy on existing Fuel master - -e Do not launch environment deployment - -F Do only create a Fuel master -+ -h Print this message and exit - -H No health check - -l Lab-name - -p Pod-name -@@ -62,6 +63,7 @@ Input parameters to the build script is: - -f Deploy on existing Fuel master - -e Do not launch environment deployment - -F Do only create a Fuel master -+-h Print this message and exit - -H Do not run fuel built in health-check after successfull deployment - -l Lab name as defined in the configuration directory, e.g. lf - -p POD name as defined in the configuration directory, e.g. pod-1 -@@ -116,7 +118,7 @@ DRY_RUN=0 - ############################################################################ - # BEGIN of main - # --while getopts "b:B:dfFHl:p:s:S:i:h:e" OPTION -+while getopts "b:B:dfFHl:p:s:S:i:he" OPTION - do - case $OPTION in - b) diff --git a/patches/opnfv-fuel/0023-deploy.py-add-multiple-bridges-support.patch b/patches/opnfv-fuel/0023-deploy.py-add-multiple-bridges-support.patch index 376a7221..37a888b4 100644 --- a/patches/opnfv-fuel/0023-deploy.py-add-multiple-bridges-support.patch +++ b/patches/opnfv-fuel/0023-deploy.py-add-multiple-bridges-support.patch @@ -43,8 +43,7 @@ index ff4582a..041ba2f 100755 check_file_exists(args.dha_file) if not args.cleanup_only: -@@ -343,7 +346,8 @@ def parse_arguments(): - check_file_exists(args.iso_file) +@@ -343,6 +346,7 @@ def parse_arguments(): log('Using image directory: %s' % args.storage_dir) create_dir_if_not_exists(args.storage_dir) - check_bridge(args.pxe_bridge, args.dha_file) -- 2.16.6