Skip to content

Conversation

@adoroszlai
Copy link
Contributor

What changes were proposed in this pull request?

HDDS-7307 and HDDS-7990 together cause test failure:

S3 Gateway Generate Secret                                            | FAIL |
7 != 0
...

The problem is that HA Proxy is not configured for the web admin port.

https://issues.apache.org/jira/browse/HDDS-12045

How was this patch tested?

$ cd hadoop-ozone/dist/target/ozone-2.0.0-SNAPSHOT/compose/ozonesecure
$ ./test-haproxy-s3g.sh
...
s3-encrypted :: S3 Secret Generate test                                       
==============================================================================
S3 Gateway Generate Secret                                            | PASS |
------------------------------------------------------------------------------
S3 Gateway Secret Already Exists                                      | PASS |
------------------------------------------------------------------------------
S3 Gateway Generate Secret By Username                                | PASS |
------------------------------------------------------------------------------
S3 Gateway Generate Secret By Username For Other User                 | PASS |
------------------------------------------------------------------------------
S3 Gateway Reject Secret Generation By Non-admin User                 | PASS |
------------------------------------------------------------------------------
s3-encrypted :: S3 Secret Generate test                               | PASS |
5 tests, 5 passed, 0 failed
==============================================================================
...
==============================================================================
s3-encrypted :: S3 Secret Revoke test                                         
==============================================================================
S3 Gateway Revoke Secret                                              | PASS |
------------------------------------------------------------------------------
S3 Gateway Revoke Secret By Username                                  | PASS |
------------------------------------------------------------------------------
S3 Gateway Revoke Secret By Username For Other User                   | PASS |
------------------------------------------------------------------------------
S3 Gateway Reject Secret Revoke By Non-admin User                     | PASS |
------------------------------------------------------------------------------
s3-encrypted :: S3 Secret Revoke test                                 | PASS |
4 tests, 4 passed, 0 failed
==============================================================================
...
==============================================================================
s3-encrypted :: S3 gateway web ui test                                        
==============================================================================
Check web UI                                                          | PASS |
------------------------------------------------------------------------------
s3-encrypted :: S3 gateway web ui test                                | PASS |
1 test, 1 passed, 0 failed
==============================================================================
...
==============================================================================
Web :: Smoke test for spnego with docker-compose environments.                
==============================================================================
Verify SPNEGO URLs without auth                                       | PASS |
------------------------------------------------------------------------------
Verify SPNEGO URLs with auth                                          | PASS |
------------------------------------------------------------------------------
Web :: Smoke test for spnego with docker-compose environments.        | PASS |
2 tests, 2 passed, 0 failed
==============================================================================

https://github.com/adoroszlai/ozone/actions/runs/12675689361/job/35327953173#step:6:563

Copy link
Contributor

@dombizita dombizita left a comment

Choose a reason for hiding this comment

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

Thanks for fixing this @adoroszlai!

@adoroszlai adoroszlai merged commit ae1843f into apache:master Jan 8, 2025
27 checks passed
@adoroszlai adoroszlai deleted the HDDS-12045 branch January 8, 2025 21:40
@adoroszlai
Copy link
Contributor Author

Thanks @dombizita for the review!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants