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

Test instructions for pubcloud engines #1201

Merged
merged 17 commits into from
Mar 10, 2025

Conversation

digitaltom
Copy link
Member

Description

  • Related Issue / Ticket / Trello card:

How to test

Change Type

Please select the correct option.

  • Bug Fix (a non-breaking change which fixes an issue)
  • New Feature (a non-breaking change which adds new functionality)
  • Documentation Update (a change which only updates documentation)

Checklist

Please check off each item if the requirement is met.

  • I have reviewed my own code and believe that it's ready for an external review.
  • I have provided comments for any hard-to-understand code.
  • I have documented the MANUAL.md file with any changes to the user experience.
  • If my changes are non-trivial, I have added a changelog entry to notify users at package/obs/rmt-server.changes.

Review

Please check out our review guidelines
and get in touch with the author to get a shared understanding of the change.

Sorry, something went wrong.

@digitaltom digitaltom force-pushed the testing_doc_for_pubcloud_engines branch from abaab35 to a4782ac Compare August 9, 2024 15:23
@digitaltom digitaltom changed the title [WIP] Test instructions for pubcloud engines Test instructions for pubcloud engines Aug 21, 2024
@felixsch
Copy link
Contributor

@digitaltom Whats the current state here? I'm not up to date?

@digitaltom
Copy link
Member Author

To be tested by @jesusbv , maybe this pr got dysfuntional by recent changes in master..

@digitaltom digitaltom removed the 2.20 label Dec 17, 2024
@felixsch
Copy link
Contributor

@jesusbv @digitaltom What needs to be done to make this work again? Otherwise I would close this here (it can be re-opened if there is progress again).

@digitaltom digitaltom force-pushed the testing_doc_for_pubcloud_engines branch from 5c29969 to 2e750fa Compare January 16, 2025 14:00
@digitaltom digitaltom force-pushed the testing_doc_for_pubcloud_engines branch from 2e750fa to 4261a37 Compare January 16, 2025 14:17
@digitaltom digitaltom force-pushed the testing_doc_for_pubcloud_engines branch from 2a581af to b8f64ff Compare January 16, 2025 14:34
@digitaltom
Copy link
Member Author

Branch is updated & fixed. @jesusbv did you try this out, can we merge it?

@felixsch
Copy link
Contributor

@jesusbv can you please have a look? It is now quiet some time. It only takes a few minutes :)

@jesusbv
Copy link
Collaborator

jesusbv commented Mar 10, 2025

I started the steps again and I get the same error every time

> bin/rmt-cli products enable SLES/15.5/x86_64
No product found for target SLES/15.5/x86_64.
Product SLES/15.5/x86_64 could not be found and was not enabled.

@jesusbv
Copy link
Collaborator

jesusbv commented Mar 10, 2025

maybe we should include

bin/rmt-cli sync

to the steps

@jesusbv
Copy link
Collaborator

jesusbv commented Mar 10, 2025

LGTM

Although, trying to register LTSS and following the steps, I got an error (that does not seem to interfere with the test)

D, [2025-03-10T12:41:34.467616 #25599] DEBUG -- :   ↳ app/controllers/api/connect/v3/systems/products_controller.rb:175:in `check_base_product_dependencies'
D, [2025-03-10T12:41:34.468502 #25599] DEBUG -- :   CACHE Product Load (0.0ms)  SELECT `products`.* FROM `products` WHERE `products`.`identifier` = 'sle-module-live-patching' AND `products`.`version` = '15.6' AND `products`.`arch` = 'x86_64' LIMIT 1
D, [2025-03-10T12:41:34.471731 #25599] DEBUG -- :   Product Load (1.1ms)  SELECT DISTINCT `products`.* FROM `products` INNER JOIN `services` ON `products`.`id` = `services`.`product_id` INNER JOIN `activations` ON `services`.`id` = `activations`.`service_id` WHERE `activations`.`system_id` = 6434 AND `products`.`product_type` = 'base' LIMIT 1
I, [2025-03-10T12:41:34.716731 #25599]  INFO -- : Completed 500 Internal Server Error in 282ms (ActiveRecord: 7.4ms | Allocations: 16046)


F, [2025-03-10T12:41:34.719229 #25599] FATAL -- :   
OpenSSL::SSL::SSLError (SSL_connect returned=1 errno=0 state=error: certificate verify failed (unable to get local issuer certificate)):
...

I, [2025-03-10T12:41:51.765230 #25599]  INFO -- : Started GET "/connect/repositories/installer" for 192.168.1.55 at 2025-03-10 12:41:51 +0000
I, [2025-03-10T12:41:51.766028 #25599]  INFO -- : Processing by Api::Connect::V4::Repositories::InstallerController#index as JSON
I, [2025-03-10T12:41:51.766050 #25599]  INFO -- :   Parameters: {"installer"=>{}}
I, [2025-03-10T12:41:51.766895 #25599]  INFO -- : Rendered ActiveModel::Serializer::Null with Hash (0.07ms)
I, [2025-03-10T12:41:51.767442 #25599]  INFO -- : Completed 422 Unprocessable Entity in 1ms (Views: 0.5ms | ActiveRecord: 0.0ms | Allocations: 431)

Copy link
Collaborator

@jesusbv jesusbv left a comment

Choose a reason for hiding this comment

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

LGTM

Co-authored-by: Jesús Bermúdez Velázquez <jesus.bv@suse.com>
@digitaltom digitaltom merged commit 18ab9e9 into master Mar 10, 2025
3 checks passed
@digitaltom digitaltom deleted the testing_doc_for_pubcloud_engines branch March 10, 2025 14:40
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.

None yet

3 participants