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

ENT-9170: Added deplyment tests for built enterprise hub package (3.18) #1268

Merged
merged 4 commits into from
Aug 25, 2023

Conversation

craigcomstock
Copy link
Contributor

@craigcomstock craigcomstock commented Aug 23, 2023

  • Adjusted autogen script to have a stable length abbreviated hash
  • Added deplyment tests for built enterprise hub package
  • send details of package sha contents to stderr for debugability

with https://github.com/cfengine/nova/pull/2095

Also check the hash to ensure it is unique or fail the script.

Ticket: ENT-10419
Changelog: none
(cherry picked from commit c9d4e00)
Can be run in docker or on a plain host (vm/metal)

Also added step in docker build to get sftp cached deps packages

Requires either GH env var ssh private key or mystiko (password-store repo)

Ticket: ENT-9170
Changelog: none
(cherry picked from commit 4db36c7)
@craigcomstock
Copy link
Contributor Author

This will fail until we patch the timing changes in the tests/reporting/deploytest.py, coming from https://github.com/cfengine/nova/pull/2091

@craigcomstock
Copy link
Contributor Author

failure

# ('SHA=0a3483beae9bfb38543d86bf8cbb021a2a75d36e2c3db06b2909eda05ac1a436', 34, 'OK')

shouldn't have happened if nova 3.18.x was correct. checking logs.

@craigcomstock
Copy link
Contributor Author

github lag, try again in an hour :(

Ticket: ENT-9170
Changelog: none
(cherry picked from commit 2c95913)
@craigcomstock craigcomstock merged commit 0c4d8bd into 3.18.x Aug 25, 2023
2 checks passed
@craigcomstock craigcomstock deleted the ENT-9170/3.18 branch August 25, 2023 16:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants