Skip to content

Conversation

@tdawson
Copy link
Member

@tdawson tdawson commented Dec 22, 2025

Content Resolver runs refresh.sh in a loop.
In order to run c10s on a seperate machine, we need two new versions of refresh.sh.

  • refresh-eln.sh - This is basically the same as the original refresh.sh but it runs content resolver with --labels eln,eln-extras
  • refresh-c10s.sh - Very close to refresh.sh but with --labels c10s and it syncs to the /c10s/ directory.

@tdawson tdawson requested a review from yselkowitz December 22, 2025 22:22
@tdawson
Copy link
Member Author

tdawson commented Dec 22, 2025

Until this get's merged, the current Content Resolver is going to wipe out the /c10s/ directory each time it finishes. That's not too much of a problem, it should give us several hours to see how things look.

@yselkowitz
Copy link
Member

What would the plan be for when/while we branch c11s? There will certainly be some overlap when both c10s and c11s are being tracked in CR (and with them being in a separate instance, and CS providing only one snapshot per day, perhaps they could be allowed to overlap even longer). Would you make yet another instance for c11s, or use a single instance for all active CS versions? If the latter, the a location should be more generic than /c10s/.

@tdawson
Copy link
Member Author

tdawson commented Dec 22, 2025

I was thinking of having a separate machine for c11s, which would be in the /c11s/ directory.
Although the machine I put this on could handle both c10s and c11s, so having them both run on one machine is a possibility.
I agree, if we have them both running on the same machine, we should have a more generic directory. Maybe /cs/ ?

@tdawson
Copy link
Member Author

tdawson commented Dec 22, 2025

It looks like it works. Ya!!
https://tiny.distro.builders/c10s/views.html

Copy link
Member

@yselkowitz yselkowitz left a comment

Choose a reason for hiding this comment

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

Are you planning on deploying the CS instance fairly soon after merging this? If so, let's merge the ELN changes into refresh.sh and add only the CS script.

A dedicated CS instance should be able to handle multiple versions, particularly given that CS composes are (only) nightly. Do you think we really need separate instances for each version, and how much harder will that make standing up c11s when the time comes?

@tdawson
Copy link
Member Author

tdawson commented Dec 23, 2025

Are you planning on deploying the CS instance fairly soon after merging this? If so, let's merge the ELN changes into refresh.sh and add only the CS script.

Yes, I would like to have this running by end of work day today.
That sounds good. I'll make that change.

A dedicated CS instance should be able to handle multiple versions, particularly given that CS composes are (only) nightly. Do you think we really need separate instances for each version, and how much harder will that make standing up c11s when the time comes?

I don't think we need separate instances. One machine should be able to do them both. Especially since they don't have to do extras. I'll make a change so that this goes into /cs/ then when c11s comes around, we just need to add it's tag to the labels option.

I think standing up a new instance this size would require fedora infra to set it up. For this project, I lucked out because Adam had a second machine he never shut down.

@tdawson
Copy link
Member Author

tdawson commented Dec 23, 2025

Changes made.
Let me know what you think. If everything is good I'll do a rebase so the git commit is cleaner.

  (Currently c10s)
Update refresh.sh to only do eln labels
  (Currently eln and eln-extras)
Update refresh.sh to exclude the /cs/ directory

Signed-off-by: Troy Dawson <tdawson@redhat.com>
@tdawson tdawson merged commit d16faff into fedora-eln:master Dec 23, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants