Skip to content

Initiate Jenkins pipeline#2840

Merged
troydai merged 3 commits intoAzure:masterfrom
troydai:jenkins
Apr 12, 2017
Merged

Initiate Jenkins pipeline#2840
troydai merged 3 commits intoAzure:masterfrom
troydai:jenkins

Conversation

@troydai
Copy link
Copy Markdown
Contributor

@troydai troydai commented Apr 12, 2017

  1. Add build task
  2. Add basic performance test task
  3. Add performance measure script

This checklist is used to make sure that common guidelines for a pull request are followed.

General Guidelines

  • The PR has modified HISTORY.rst with an appropriate description of the change (see Modifying change log).

Command Guidelines

  • Each command and parameter has a meaningful description.
  • Each new command has a test.

(see Authoring Command Modules)

1. Add build task
2. Add basic performance test task
3. Add performance measure script
@codecov-io
Copy link
Copy Markdown

codecov-io commented Apr 12, 2017

Codecov Report

Merging #2840 into master will increase coverage by 0.04%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2840      +/-   ##
==========================================
+ Coverage   62.88%   62.92%   +0.04%     
==========================================
  Files         464      464              
  Lines       25901    25899       -2     
  Branches     3944     3943       -1     
==========================================
+ Hits        16288    16298      +10     
+ Misses       8594     8567      -27     
- Partials     1019     1034      +15
Impacted Files Coverage Δ
...ure-cli-core/azure/cli/core/commands/parameters.py 70.96% <0%> (-2.16%) ⬇️
...etwork/azure/cli/command_modules/network/custom.py 63.46% <0%> (+1.08%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a8395ea...5cf7cc2. Read the comment docs.

echo "Version number: $version"

if [ -d /var/build_share ]; then
echo 'Directory /var/build_share is found. The artifacts will be archieved there.'
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

typo archieved

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

will update.


echo 'Build Azure CLI and its command modules '

pip install -r requirements.txt
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

You shouldn't need to install all the requirements just to build.
pip install --upgrade wheel should be sufficient and this is required because you use bdist_wheel below.

echo "Install build from $build_folder"

python -m pip install azure-cli --find-links file://$build_folder/artifacts/build -v
python -m pip list
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Would suggest pip freeze instead. That output can be easily saved in a file and used later whilst pip list is only user friendly.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

The goal is to print the dependencies in log for human consumption. You think this is good?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

$ pip list
adal (0.4.3)
appdirs (1.4.3)
applicationinsights (0.10.0)
argcomplete (1.8.0)
asn1crypto (0.22.0)
$ pip freeze
adal==0.4.3
appdirs==1.4.3
applicationinsights==0.10.0
argcomplete==1.8.0
asn1crypto==0.22.0

They are both readable but advantage of pip freeze is you can use the output of that into pip install -r ...

@troydai
Copy link
Copy Markdown
Contributor Author

troydai commented Apr 12, 2017

@derekbekoe updated.

Copy link
Copy Markdown
Member

@derekbekoe derekbekoe left a comment

Choose a reason for hiding this comment

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

LGTM!

@troydai troydai merged commit a198a63 into Azure:master Apr 12, 2017
@troydai troydai deleted the jenkins branch April 12, 2017 22:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants