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

[fix][io] Update test certs for Elasticsearch #21001

Merged

Conversation

michaeljmarshall
Copy link
Member

Fixes: #20997

Motivation

Update the expired certs to get tests passing.

Modifications

  • Update all certs. See README.md in files for detailed steps.

Verifying this change

This change is covered by tests.

Documentation

  • doc-not-needed

Copy link
Contributor

@dlg99 dlg99 left a comment

Choose a reason for hiding this comment

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

LGTM

@codecov-commenter
Copy link

Codecov Report

Merging #21001 (f0ed4bb) into master (a0372f4) will increase coverage by 38.29%.
Report is 20 commits behind head on master.
The diff coverage is n/a.

Impacted file tree graph

@@              Coverage Diff              @@
##             master   #21001       +/-   ##
=============================================
+ Coverage     34.80%   73.09%   +38.29%     
+ Complexity    12089     3753     -8336     
=============================================
  Files          1697     1875      +178     
  Lines        129787   139443     +9656     
  Branches      14157    15333     +1176     
=============================================
+ Hits          45166   101924    +56758     
+ Misses        78629    29451    -49178     
- Partials       5992     8068     +2076     
Flag Coverage Δ
inttests 24.16% <ø> (?)
systests 25.08% <ø> (-0.08%) ⬇️
unittests 72.38% <ø> (+40.31%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

see 1453 files with indirect coverage changes

@michaeljmarshall michaeljmarshall merged commit d6734b7 into apache:master Aug 16, 2023
47 checks passed
@michaeljmarshall michaeljmarshall deleted the fix-elastsearch-tests branch August 16, 2023 04:43
michaeljmarshall added a commit that referenced this pull request Aug 16, 2023
Fixes: #20997

### Motivation

Update the expired certs to get tests passing.

### Modifications

* Update all certs. See `README.md` in files for detailed steps.

### Verifying this change

This change is covered by tests.

### Documentation

- [x] `doc-not-needed`

(cherry picked from commit d6734b7)
michaeljmarshall added a commit that referenced this pull request Aug 16, 2023
Fixes: #20997

### Motivation

Update the expired certs to get tests passing.

### Modifications

* Update all certs. See `README.md` in files for detailed steps.

### Verifying this change

This change is covered by tests.

### Documentation

- [x] `doc-not-needed`

(cherry picked from commit d6734b7)
michaeljmarshall added a commit to michaeljmarshall/pulsar that referenced this pull request Aug 16, 2023
Fixes: apache#20997

### Motivation

Update the expired certs to get tests passing.

### Modifications

* Update all certs. See `README.md` in files for detailed steps.

### Verifying this change

This change is covered by tests.

### Documentation

- [x] `doc-not-needed`

(cherry picked from commit d6734b7)
michaeljmarshall added a commit that referenced this pull request Aug 16, 2023
Fixes: #20997

Update the expired certs to get tests passing.

* Update all certs. See `README.md` in files for detailed steps.

This change is covered by tests.

- [x] `doc-not-needed`

(cherry picked from commit d6734b7)
michaeljmarshall added a commit to datastax/pulsar that referenced this pull request Aug 16, 2023
Fixes: apache#20997

### Motivation

Update the expired certs to get tests passing.

### Modifications

* Update all certs. See `README.md` in files for detailed steps.

### Verifying this change

This change is covered by tests.

### Documentation

- [x] `doc-not-needed`

(cherry picked from commit d6734b7)
@Technoboy-
Copy link
Contributor

Leave another solution:

1. https://www.elastic.co/guide/en/elasticsearch/reference/current/targz.html
2. bin/elasticsearch-certutil ca --pem --out cacert.zip --days 36500 -s --ca-dn "CN=localhost"
3. unzip cacert.zip 
4. in/elasticsearch-certutil cert --ca-cert cacert/cacert.crt --ca-key cacert/cacert.key --pem --name localhost  --out localhost.zip --days 36500 -s
5. cd localhost
6. openssl x509 -in cacert/localhost.crt -out elasticsearch.pem
7. keytool -import -alias truststore -file localhost.crt -keystore truststore.jks
  - password: changeit
8. keytool -import -alias keystore -file localhost.crt -keystore keystore.jks   
  - password: changeit
9. mv localhost.crt elasticsearch.crt
     mv localhost.key elasticsearch.key 
10. mv ./ cacert/.* .

michaeljmarshall added a commit that referenced this pull request Aug 24, 2023
Fixes: #20997

### Motivation

Update the expired certs to get tests passing.

### Modifications

* Update all certs. See `README.md` in files for detailed steps.

### Verifying this change

This change is covered by tests.

### Documentation

- [x] `doc-not-needed`

(cherry picked from commit d6734b7)
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.

Flaky-test: ElasticSearch8ClientSslTests.testSslBasic
5 participants