Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Release v0.13.1 #924

Merged
merged 28 commits into from
Sep 29, 2020
Merged

Release v0.13.1 #924

merged 28 commits into from
Sep 29, 2020

Conversation

ccarterlandis
Copy link
Contributor

@ccarterlandis ccarterlandis commented Sep 21, 2020

Release v0.13.1

Changelog

  • Refactor add-github-org implementation
  • Add ability to create repo groups from GitHub orgs using the Augur CLI
  • Fixed a few documentation links and avoided using the word 'here' as a link
  • Add sudo warning to installation docs
  • Hide password during config setup process
  • Update installation logging for clarity
  • Update default config file location
  • Update augur util kill usage
  • Add augur util stop
  • Revert "updating augur util kill cli_kill_processes and kill_processes to send SIGKILL after waiting 15 seconds"
  • Remove duplicated and outdated doc pages
  • Update Travis CI build to install all worker deps
  • Add tag_only releases to the release worker

ivanayov and others added 21 commits August 24, 2020 22:02
This updates release worker to insert new values when release_id is not
already inserted and update when a row exist. Includes few tiny fixes

Signed-off-by: Ivana Yovcheva <iyovcheva@vmware.com>
This change checks for and adds tags to the `releases` table when they
are used instead of actual GitHub releases. It also adds metrics for
tag_only releases

Signed-off-by: Ivana Yovcheva <iyovcheva@vmware.com>
Add tag_only releases to the release worker
Signed-off-by: Carter Landis <c@carterlandis.com>
Signed-off-by: Carter Landis <c@carterlandis.com>
…ses to send SIGKILL after waiting 15 seconds and checking what is still running again."

This reverts commit 7826039.
Signed-off-by: Carter Landis <c@carterlandis.com>
Signed-off-by: Carter Landis <c@carterlandis.com>
Fix Augur process control commands
Currently, on all branches, using `augur configure generate` (the augur lib command to create a config file) behaves in an undesirable way if installed using `make install`. The issue is that after running `augur configure generate`, if you used `make install` to install Augur, your config file is not written to root of your cloned source code directory. It is instead written to the `site-packages/augur/` folder that lives with your Python binary inside your virtualenv folder. The reason that it writes to this location is due to how I implemented the config generation script. The script uses a package-level variable called `ROOT_AUGUR_DIRECTORY` to determine the location of the root of the file currently that is currently running; when installed in non-editable mode (without using the `-e` flag, like in `make install`), `pip` will copy ONLY the files defined in your `MANIFEST.in` from your cloned source code directory (if that's what you're installing) into that `site-packages` folder; **these are the files that are "running" when using augur installed in non-editable mode** - that is, without `-e`.

However, when installing the package in editable mode with `-e` (like `pip install -e .` in `make install-dev`), the files are **not** copied into that folder. Instead, some stuff happens behind the scenes, and `pip` links the contents of your local cloned directory to the `site-packages` folder; this is what allows you to make changes, save, and then run the commands again. That means when we use `install-dev`, **these are the files that are running when we execute augur commands. This is why the augur.config.json file is always installed to your cloned source code directory; that's the "root" of the currently running Augur binary.

There's a very important caveat to using `make install`, however; _it has no way of knowing where you installed the binary from_. This means it has _no way of installing the config file to your cloned source code directory_. In fact, it's not even guranteed that the source is on your machine; all the other packages you install with `pip` like `pandas` only have the files in their `MANIFEST.in` copied as well. If we want the config file to only ever be in one location, then this is problem; it can't be in the source code directory like it has been. We need to pick a standard location on the file system to store the config file outside the cloned source code tree.

To this end, this patch changes the default config file location `$HOME/.augur/augur.config.json`.
However, if it detects a config file in the root Augur directory, then it will use that instead.

Signed-off-by: Carter Landis <c@carterlandis.com>
Signed-off-by: Carter Landis <c@carterlandis.com>
Update default config file location
This patch exposes the logs for the dependency installation
process, including all errors. It also moves all logs from
logs/install to just logs/.

Signed-off-by: Carter Landis <c@carterlandis.com>
Signed-off-by: Carter Landis <c@carterlandis.com>
Signed-off-by: Carter Landis <c@carterlandis.com>
Signed-off-by: Dawn M. Foster <fosterd@vmware.com>
@ccarterlandis ccarterlandis added feature-request Request for a new feature in Augur installation Related the installation process documentation Updates documentation usability Usability related issues (bugs, features, etc) CLI Related to Augur's CLI labels Sep 21, 2020
@ccarterlandis ccarterlandis self-assigned this Sep 21, 2020
Signed-off-by: Carter Landis <c@carterlandis.com>
sgoggins
sgoggins previously approved these changes Sep 23, 2020
docs/README.md Show resolved Hide resolved
ccarterlandis and others added 3 commits September 23, 2020 11:55
Signed-off-by: Carter Landis <c@carterlandis.com>
Add ability to create repo groupgs from GH orgs on the CLI
@ccarterlandis ccarterlandis marked this pull request as ready for review September 28, 2020 18:29
@ccarterlandis
Copy link
Contributor Author

@sgoggins & @gabe-heim this PR is ready for review

Copy link
Contributor

@gabe-heim gabe-heim left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Member

@sgoggins sgoggins left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. There is another PR that provides a needed schema update that I would like in the current release, as it addresses an error in the release worker.

@sgoggins
Copy link
Member

@ccarterlandis and @gabe-heim : Good to merge?

@sgoggins sgoggins merged commit 29c54ce into master Sep 29, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLI Related to Augur's CLI documentation Updates documentation feature-request Request for a new feature in Augur installation Related the installation process usability Usability related issues (bugs, features, etc)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants