Skip to content

Conversation

tlacroix
Copy link

Issue #, if available: #185 - ref.: AWSGLUE-713

Description of changes:
Simply added an error trapping directive and ignoring for the failing rmdir.

Makes codedeploy-agent compatible with Ubuntu 18.04 when deploying two different apps consecutively in the same base directory. ie.:

  • /var/www/some-api/
  • /var/www/some-angular-app/

This works fine on Ubuntu 16.04, but you get the following error on Ubuntu 18.04 with Ruby 2.5:
Unknown error, Directory not empty @ dir_s_rmdir - /var/www

It might not be the best way to do it, but it is meant to provide a solution until this officially gets fixed since it has been open since August 2018 already. It's in line with the AWSGLUE-713 comment, and we've been successfully using this patch in development and production since then.

--

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Makes codedeploy-agent compatible with Ubuntu 18.04 when deploying two different apps in the base common directory. ie.:
- `/var/www/some-api/`
- `/var/www/some-angular-app/`

This works fine on Ubuntu 16.04, but you get the following error on Ubuntu 18.04 with ruby 2.5:
`Unknown error, Directory not empty @ dir_s_rmdir - /var/www`

It might not be the best way to do it, but it is meant to provide a solution until this officially gets fixed since it has been open since August 2018 already.
@coveralls
Copy link

Coverage Status

Coverage remained the same at 92.095% when pulling 817f620 on tlacroix:fix-unknown-error-directory-not-empty-dir_s_rmdir into 866632b on aws:master.

@woodhull
Copy link

woodhull commented Jan 5, 2020

We have the same rmdir issue, after upgrading to ubuntu 18.04 from 16.04. This seems for us to be unrelated to running multiple apps on the same machine.

We get the same error when trying to delete a directory that is part of our app:

Directory not empty @ dir_s_rmdir - /srv/agra/public

@Helen1987
Copy link
Contributor

Close the CR in favor of #201

@Helen1987 Helen1987 closed this Feb 28, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants