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] Porting #51840 to master #54679

Merged
merged 17 commits into from
Oct 5, 2022

Conversation

garethgreenaway
Copy link
Contributor

@garethgreenaway garethgreenaway commented Sep 19, 2019

Porting #51840 to master

@garethgreenaway garethgreenaway requested a review from a team September 19, 2019 01:18
@ghost ghost requested review from Akm0d and removed request for a team September 19, 2019 01:18
@garethgreenaway garethgreenaway changed the title [2019.2.1] Porting #51840 to 2019.2.1 [master] Porting #51840 to master Sep 24, 2019
@garethgreenaway garethgreenaway changed the base branch from 2019.2.1 to master September 24, 2019 22:33
@codecov
Copy link

codecov bot commented Sep 25, 2019

Codecov Report

Merging #54679 into master will increase coverage by 21.7%.
The diff coverage is 28.57%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master   #54679      +/-   ##
==========================================
+ Coverage   15.07%   36.78%   +21.7%     
==========================================
  Files        1577     1578       +1     
  Lines      270239   270274      +35     
  Branches    57591    56208    -1383     
==========================================
+ Hits        40744    99413   +58669     
+ Misses     225624   159462   -66162     
- Partials     3871    11399    +7528
Flag Coverage Δ
#archlts ?
#centos7 ?
#debian8 ?
#py2 36.78% <28.57%> (+23.39%) ⬆️
#py3 ?
#tcp 36.78% <28.57%> (?)
#ubuntu1604 36.78% <28.57%> (+31.76%) ⬆️
#zeromq ?
Impacted Files Coverage Δ
salt/roster/sshknownhosts.py 28.57% <28.57%> (ø)
salt/proxy/dummy.py 0% <0%> (-80.87%) ⬇️
salt/proxy/docker.py 0% <0%> (-62.5%) ⬇️
salt/proxy/chronos.py 0% <0%> (-50%) ⬇️
salt/proxy/marathon.py 0% <0%> (-50%) ⬇️
salt/transport/frame.py 30.3% <0%> (-46.97%) ⬇️
salt/proxy/cisconso.py 0% <0%> (-46.43%) ⬇️
salt/proxy/arista_pyeapi.py 0% <0%> (-45.66%) ⬇️
salt/proxy/nxos_api.py 0% <0%> (-44.74%) ⬇️
salt/utils/openstack/pyrax/authenticate.py 0% <0%> (-41.67%) ⬇️
... and 1194 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 e2b86bf...56a8588. Read the comment docs.

@dwoz dwoz requested a review from a team as a code owner December 21, 2019 16:25
@dwoz
Copy link
Contributor

dwoz commented Dec 23, 2019

@team-core the test failures, docs, and lint all look legit

twangboy
twangboy previously approved these changes Dec 23, 2019
@dwoz
Copy link
Contributor

dwoz commented Apr 13, 2020

Black needs to be run on this.

Copy link
Contributor

@Ch3LL Ch3LL left a comment

Choose a reason for hiding this comment

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

this needs a changelog

salt/roster/sshknownhosts.py Show resolved Hide resolved
@Ch3LL Ch3LL added the Sulfur v3006.0 release code name and version label Oct 3, 2022
Ch3LL
Ch3LL previously approved these changes Oct 4, 2022
@garethgreenaway garethgreenaway merged commit 1c4beef into saltstack:master Oct 5, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
master-port Sulfur v3006.0 release code name and version
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants