Skip to content

Initiate Jenkins pipeline - #2840

Merged
Troy Dai (troydai) merged 3 commits into
Azure:masterfrom
troydai:jenkins
Apr 12, 2017
Merged

Troy Dai (troydai) merged 3 commits into
Azure:masterfrom
troydai:jenkins

Conversation

@troydai

Copy link
Copy Markdown
Contributor
  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

Codecov (codecov-io) commented Apr 12, 2017

Copy link
Copy Markdown

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.

Comment thread scripts/jenkins_archive.sh Outdated
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.

Comment thread scripts/jenkins_build.sh Outdated

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.

Comment thread scripts/jenkins_perf.sh Outdated
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

Derek Bekoe (@derekbekoe) updated.

@derekbekoe Derek Bekoe (derekbekoe) left a comment

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.

LGTM!

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