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

[master] Consolidate static requirements #54822

Merged

Conversation

s0undt3ch
Copy link
Collaborator

@s0undt3ch s0undt3ch commented Sep 30, 2019

What does this PR do?

See title.

Commits signed with GPG?

Yes


This PR can only be merged after #54943

@s0undt3ch s0undt3ch added the ZZZWarRoom retired label War Room Issues label Sep 30, 2019
@s0undt3ch s0undt3ch changed the title [WIP] Consolidate linux static requirements [WIP][master] Consolidate linux static requirements Oct 1, 2019
@s0undt3ch s0undt3ch force-pushed the hotfix/consolidate-requirements-master branch from 852750a to feb8b96 Compare October 7, 2019 17:29
@s0undt3ch s0undt3ch requested a review from a team as a code owner October 7, 2019 17:29
@ghost ghost requested a review from cmcmarrow October 7, 2019 17:29
@s0undt3ch s0undt3ch force-pushed the hotfix/consolidate-requirements-master branch from feb8b96 to 42a70ec Compare October 8, 2019 10:34
@s0undt3ch s0undt3ch changed the title [WIP][master] Consolidate linux static requirements [WIP][master] Consolidate static requirements + Remove RAET Oct 8, 2019
@s0undt3ch s0undt3ch changed the title [WIP][master] Consolidate static requirements + Remove RAET [master] Consolidate static requirements + Remove RAET Oct 8, 2019
@s0undt3ch s0undt3ch force-pushed the hotfix/consolidate-requirements-master branch 2 times, most recently from 051ed27 to 4e4dd67 Compare October 10, 2019 11:21
@s0undt3ch s0undt3ch changed the title [master] Consolidate static requirements + Remove RAET [master] Consolidate static requirements Oct 10, 2019
@s0undt3ch s0undt3ch force-pushed the hotfix/consolidate-requirements-master branch 2 times, most recently from d8fe605 to 7d3ae0d Compare October 11, 2019 18:21
@codecov
Copy link

codecov bot commented Oct 11, 2019

Codecov Report

Merging #54822 into master will decrease coverage by 0.75%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master   #54822      +/-   ##
==========================================
- Coverage   11.77%   11.01%   -0.76%     
==========================================
  Files        1507     1507              
  Lines      251142   251142              
  Branches    53653    53653              
==========================================
- Hits        29573    27674    -1899     
- Misses     219486   221609    +2123     
+ Partials     2083     1859     -224
Flag Coverage Δ
#archlts 11.01% <ø> (-0.01%) ⬇️
#centos7 ?
#py2 10.88% <ø> (-0.77%) ⬇️
#py3 10.72% <ø> (-0.72%) ⬇️
#ubuntu1604 ?
#zeromq 11.01% <ø> (-0.76%) ⬇️
Impacted Files Coverage Δ
salt/config/schemas/esxdatacenter.py 0% <0%> (-100%) ⬇️
salt/proxy/dummy.py 0% <0%> (-80.87%) ⬇️
salt/proxy/docker.py 0% <0%> (-62.5%) ⬇️
salt/cli/caller.py 0% <0%> (-51.25%) ⬇️
salt/proxy/chronos.py 0% <0%> (-50%) ⬇️
salt/log/handlers/log4mongo_mod.py 0% <0%> (-50%) ⬇️
salt/proxy/marathon.py 0% <0%> (-50%) ⬇️
salt/proxy/cisconso.py 0% <0%> (-46.43%) ⬇️
salt/proxy/arista_pyeapi.py 0% <0%> (-45.66%) ⬇️
salt/cli/call.py 0% <0%> (-45.46%) ⬇️
... and 167 more

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 0e7ca21...7d3ae0d. Read the comment docs.

@s0undt3ch s0undt3ch force-pushed the hotfix/consolidate-requirements-master branch from 7d3ae0d to ca173f9 Compare October 14, 2019 04:59
@s0undt3ch s0undt3ch force-pushed the hotfix/consolidate-requirements-master branch from ca173f9 to 0d13fbc Compare October 17, 2019 19:22
@dwoz dwoz merged commit 22d2390 into saltstack:master Oct 18, 2019
@s0undt3ch s0undt3ch deleted the hotfix/consolidate-requirements-master branch October 18, 2019 16:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ZZZWarRoom retired label War Room Issues
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants