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

Add debian repository to views #1029

Merged
merged 1 commit into from
Feb 21, 2024

Conversation

dosas
Copy link
Collaborator

@dosas dosas commented Nov 7, 2023

to enable debian repo testing for orcharhino.

@omkarkhatavkar
Copy link

Can one of the admins verify this patch?

@JacobCallahan
Copy link
Member

@dosas since we can't really test this (aside from upstream builds), can you provide some examples of your use?

@dosas
Copy link
Collaborator Author

dosas commented Nov 15, 2023

@JacobCallahan I understand.
The only example I have is from a robottelo test that is not upstream:

        session.repository.create(
            product_name,
            {
                'name': repo_name,
                'repo_type': REPO_TYPE['deb'],  # "deb"
                'repo_content.upstream_url': repo.url,
                'repo_content.releases': deb_releases,
                'repo_content.errata_url': repo.errata_url,
            },
        )

@dosas
Copy link
Collaborator Author

dosas commented Dec 29, 2023

Any news on this?

@vijaysawant
Copy link
Contributor

trigger: test-robottelo
pytest: tests/foreman/ui/test_repository.py -k test_positive_create_in_different_orgs

@omkarkhatavkar
Copy link

trigger: test-robottelo
pytest: tests/foreman/ui/test_repository.py

@omkarkhatavkar
Copy link

These failures are unrelated to this PR

@omkarkhatavkar
Copy link

omkarkhatavkar commented Jan 3, 2024

@vijaysawant @vsedmik @sambible @ColeHiggins2 Anyone of you can confirm the Debian Repository Changes?

@ogajduse
Copy link
Member

Update: Phoenix team pinged in their team channel.

dosas pushed a commit to ATIX-AG/airgun that referenced this pull request Jan 29, 2024
dosas pushed a commit to ATIX-AG/airgun that referenced this pull request Feb 7, 2024
@dosas
Copy link
Collaborator Author

dosas commented Feb 20, 2024

Any news from the phoenix team?

@vijaysawant
Copy link
Contributor

trigger: test-robottelo
pytest: tests/foreman/ui/test_repository.py

@Satellite-QE
Copy link
Contributor

PRT Result

Build Number: 219
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/ui/test_repository.py
Test Result : =========================== short test summary info ============================

@Satellite-QE Satellite-QE added the PRT-Failed Indicates that latest PRT run is failed for the PR label Feb 21, 2024
Copy link
Contributor

@sambible sambible left a comment

Choose a reason for hiding this comment

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

Failure seems unrelated. This change would not result in any problems, and seems fine.

@omkarkhatavkar omkarkhatavkar merged commit 597175e into SatelliteQE:master Feb 21, 2024
1 of 3 checks passed
@dosas
Copy link
Collaborator Author

dosas commented Mar 12, 2024

@sambible Thanks for merging this. How do I get these changes into 6.15.z, do I have to create a separate PR?

dosas pushed a commit to ATIX-AG/airgun that referenced this pull request Mar 12, 2024
@vsedmik
Copy link
Contributor

vsedmik commented Mar 14, 2024

@dosas The standard/easiest way is just adding CherryPick and your desired 6.<Y>.z labels before merging, so it gets auto cherry-picked. Since this PR is merged already I would suggest raising a new PR against the 6.15.z branch, cherry-picking your 03a97c8 commit.

@dosas
Copy link
Collaborator Author

dosas commented Mar 14, 2024

@vsedmik Thanks, but the problem is I cannot add labels.

I will create another PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PRT-Failed Indicates that latest PRT run is failed for the PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants