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
Improvement/bump salt 3006.6 #4237
base: development/127.0
Are you sure you want to change the base?
Conversation
Hello eg-ayoub,My role is to assist you with the merge of this Status report is not available. |
Waiting for approvalThe following approvals are needed before I can proceed with the merge:
Peer approvals must include at least 1 approval from the following list: |
mock == 3.0.5 | ||
parameterized == 0.7.4 | ||
etcd3 != 0.11.0 | ||
kubernetes == 18.20.0 | ||
urllib3 | ||
pyfakefs | ||
psutil | ||
# because etcd3 package |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
maybe find another etcd lib
images/salt-master/Dockerfile
Outdated
&& dnf install -y git \ | ||
&& pip3 install "git+https://github.com/kubernetes-client/python.git@cef5e9bd10a6d5ca4d9c83da46ccfe2114cdaaf8#egg=kubernetes" \ | ||
&& salt-pip install "git+https://github.com/kubernetes-client/python.git@aab72fdeb4122337e0e3e27ad4b489dcdb7bd00d#egg=kubernetes" \ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
use tag instead v27.2.0
images/salt-master/Dockerfile
Outdated
&& pip3 install "pip == 20.1" \ | ||
&& pip3 install "protobuf ~= 3.19.4" "etcd3 != 0.11.0" \ | ||
&& dnf install -y salt-master salt-api salt-ssh openssh-clients procps-ng \ | ||
&& salt-pip install "protobuf ~= 3.19.4" "etcd3 != 0.11.0" \ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
protobuf <= 3.20.1 from tests
Waiting for approvalThe following approvals are needed before I can proceed with the merge:
Peer approvals must include at least 1 approval from the following list: |
Waiting for approvalThe following approvals are needed before I can proceed with the merge:
Peer approvals must include at least 1 approval from the following list: |
c0e6cf7
to
c084c49
Compare
Waiting for approvalThe following approvals are needed before I can proceed with the merge:
Peer approvals must include at least 1 approval from the following list: |
Waiting for approvalThe following approvals are needed before I can proceed with the merge:
Peer approvals must include at least 1 approval from the following list: |
Waiting for approvalThe following approvals are needed before I can proceed with the merge:
Peer approvals must include at least 1 approval from the following list: |
Waiting for approvalThe following approvals are needed before I can proceed with the merge:
Peer approvals must include at least 1 approval from the following list: |
fc81652
to
af048a1
Compare
Waiting for approvalThe following approvals are needed before I can proceed with the merge:
Peer approvals must include at least 1 approval from the following list: |
bd0c7f6
to
6832aea
Compare
Waiting for approvalThe following approvals are needed before I can proceed with the merge:
Peer approvals must include at least 1 approval from the following list: |
6832aea
to
2c14260
Compare
Waiting for approvalThe following approvals are needed before I can proceed with the merge:
Peer approvals must include at least 1 approval from the following list: |
7b76412
to
ff714ba
Compare
Waiting for approvalThe following approvals are needed before I can proceed with the merge:
Peer approvals must include at least 1 approval from the following list: |
Waiting for approvalThe following approvals are needed before I can proceed with the merge:
Peer approvals must include at least 1 approval from the following list: |
Waiting for approvalThe following approvals are needed before I can proceed with the merge:
Peer approvals must include at least 1 approval from the following list: |
f355038
to
adf2e08
Compare
Waiting for approvalThe following approvals are needed before I can proceed with the merge:
Peer approvals must include at least 1 approval from the following list: |
Version 7.0.0 drops usage of 'style' key which we use in our theme cf. sphinx-doc/sphinx#11381
saltstack/salt#65692 temporarily until the fix for this bug is released in 3006.7
(cherry picked from commit 7363b9e)
- Remove m2crypto - Remove now empty epel repo - Edit states to match the new x509_v2 modules and states
This reverts commit ff714ba.
2661cc6
to
293f1f0
Compare
Request integration branchesWaiting for integration branch creation to be requested by the user. To request integration branches, please comment on this pull request with the following command:
Alternatively, the |
WIP