box / box/wavectl

The git integration in pull can have better commit messages using the resource history from wavefront server

Open
#18 0 comments 0 reactions 0 assignees View on GitHub
enhancement
Dominant language
Python
Stars
16
Forks
10
PR merge metrics
No merged PRs in 30d

Description

Right now the git integration with the pull command results in commit messages like this

```
hbaba@mbp-005063:~/misc/wavefront ((HEAD detached at d638aa8))$ git lola
* dbee3c1 () [2018-01-20 23:46:46 -0800] (master, 2018-01-20--23-48-23-pull-branch, 2018-01-20--23-47-43-pull-branch, 2018-01-20--23-46-34-pull-branch) Added files due to pull cmd:/Users/hbaba/box/src/skynet/wavectl/wavectl/main.py pull --inGit /Users/hbaba/misc/wavefront alert
* d638aa8 () [2018-01-20 23:37:10 -0800] (HEAD, 2018-01-20--23-37-00-pull-branch) Added files due to pull cmd:/Users/hbaba/box/src/skynet/wavectl/wavectl/main.py pull --inGit /Users/hbaba/misc/wavefront alert
* e8def81 () [2018-01-20 23:36:12 -0800] (2018-01-20--23-35-57-pull-branch) Added files due to pull cmd:/Users/hbaba/box/src/skynet/wavectl/wavectl/main.py pull --inGit /Users/hbaba/misc/wavefront alert
* e570f08 () [2018-01-20 23:28:26 -0800] (2018-01-20--23-28-15-pull-branch) Added files due to pull cmd:/Users/hbaba/box/src/skynet/wavectl/wavectl/main.py pull --inGit /Users/hbaba/misc/wavefront alert
* 443b4b8 () [2018-01-20 23:27:04 -0800] (2018-01-20--23-26-49-pull-branch) Added files due to pull cmd:/Users/hbaba/box/src/skynet/wavectl/wavectl/main.py pull --inGit /Users/hbaba/misc/wavefront alert
* 380738b () [2018-01-20 23:25:17 -0800] (2018-01-20--23-25-04-pull-branch) Added files due to pull cmd:/Users/hbaba/box/src/skynet/wavectl/wavectl/main.py pull --inGit /Users/hbaba/misc/wavefront alert
* 539efc1 () [2018-01-20 23:23:48 -0800] (2018-01-20--23-23-33-pull-branch) Added files due to pull cmd:/Users/hbaba/box/src/skynet/wavectl/wavectl/main.py pull --inGit /Users/hbaba/misc/wavefront alert
* f504619 () [2018-01-20 23:23:33 -0800] (1970-01-01--00-00-00-pull-branch) Initial commit with the README.md file```

This is not very useful to the users.

The v2 api of Wavefront has a "versions" endpoint. There the time and the author for each change to each resource can be seen.

It would be very very cool if the generated git history could read the version history for each resoruce from wavefront and mention the authors, and the modification times in the commit messages.

With that the resulting git repo in `pull --inGit ` command would be much more useful.

Contributor guide

Open the contributing guide

Research direction

Start by tracing the pull --inGit command and the generated git history, then inspect the Wavefront v2 versions endpoint described in the issue. Done means generated commit messages include each resource change's author and modification time instead of only the generic pull command text.

Written by the indexing model from the issue text.

Assessment

Tech stack
git, python
Domain
api, cli
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.