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

WIP: config-bot: add bump-lockfiles functionality #48

Closed
wants to merge 1 commit into from

Conversation

jlebon
Copy link
Member

@jlebon jlebon commented Oct 8, 2019

This is meant to address a smarter way to bump lockfiles.
See: coreos/fedora-coreos-tracker#293

Only supports git push for now. Working on making that smarter.

This is meant to address a smarter way to bump lockfiles.
See: coreos/fedora-coreos-tracker#293

Only supports `git push` for now. Working on making that smarter.
@@ -1,4 +1,4 @@
FROM registry.fedoraproject.org/fedora:30
FROM quay.io/coreos-assembler/coreos-assembler:master
Copy link
Member

Choose a reason for hiding this comment

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

Hmm. Things just got a lot more heavyweight :) - Does this mean we also require /dev/kvm for config-bot now?

Copy link
Member Author

Choose a reason for hiding this comment

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

I originally wanted to use cosa for this, and had a patch to drop the req. on /dev/kvm for our needs. :) But in the end, I think it's simpler to just invoke rpm-ostree ourselves directly.

@dustymabe
Copy link
Member

Could we just make a separate pipeline that uses bodhi-updates as the source and just runs fetch --update-lockfiles and then config bot picks up that output?

IOW we keep promote-lockfiles and run the fetch --update-lockfiles somewhere else rather than as part of config bot

@jlebon
Copy link
Member Author

jlebon commented Oct 10, 2019

Could we just make a separate pipeline that uses bodhi-updates as the source and just runs fetch --update-lockfiles and then config bot picks up that output?

That could work, though hmm... feels like overkill if we can just run that one rpm-ostree command here.

jlebon added a commit to jlebon/fedora-coreos-config that referenced this pull request Nov 8, 2019
As discussed in coreos#104,
we don't actually want the bodhi-updates stream to pull in newer
packages from the pool.

That said, bodhi-updates also current acts as our lockfile updater in
testing-devel, so it's crucial that it keeps working. We're working to
decouple that, see:

coreos/fedora-coreos-tracker#293
coreos/fedora-coreos-releng-automation#48

But for now, the updates must flow...
@jlebon
Copy link
Member Author

jlebon commented Nov 8, 2019

I think an MVP for this is just doing the PR workflow so we're still covered by CI. Do a dumb detection if a PR is already opened, and just update that one if so.

We can then refine it later on to do the "branch push and silently merge if CI passes and it's trivial package bumps, otherwise open a PR".

jlebon added a commit to coreos/fedora-coreos-config that referenced this pull request Nov 8, 2019
As discussed in #104,
we don't actually want the bodhi-updates stream to pull in newer
packages from the pool.

That said, bodhi-updates also current acts as our lockfile updater in
testing-devel, so it's crucial that it keeps working. We're working to
decouple that, see:

coreos/fedora-coreos-tracker#293
coreos/fedora-coreos-releng-automation#48

But for now, the updates must flow...
@dustymabe
Copy link
Member

Sounds good

@jlebon
Copy link
Member Author

jlebon commented Apr 7, 2020

Going to pick this up again soon!

jlebon added a commit to jlebon/fedora-coreos-pipeline that referenced this pull request May 1, 2020
This job will implement lockfile bumping for testing-devel and
next-devel: coreos/fedora-coreos-tracker#293.

The original plan for this functionality was to have it in config-bot:
coreos/fedora-coreos-releng-automation#48

But in the end, I think it's more natural to have it as a Jenkins job
given that it does a lot of the same things as the pipeline/upstream CI
jobs. So that way it looks and feels just like another job that runs
cosa, and we get kola artifacts, we can re-use the shared library,
it's easily inspectable, we can hook it to Slack, etc...
jlebon added a commit to jlebon/fedora-coreos-pipeline that referenced this pull request May 1, 2020
This job will implement lockfile bumping for testing-devel and
next-devel: coreos/fedora-coreos-tracker#293.

The original plan for this functionality was to have it in config-bot:
coreos/fedora-coreos-releng-automation#48

But in the end, I think it's more natural to have it as a Jenkins job
given that it does a lot of the same things as the pipeline/upstream CI
jobs. So that way it looks and feels just like another job that runs
cosa, and we get kola artifacts, we can re-use the shared library,
it's easily inspectable, we can hook it to Slack, etc...
jlebon added a commit to jlebon/fedora-coreos-pipeline that referenced this pull request May 1, 2020
This job will implement lockfile bumping for testing-devel and
next-devel: coreos/fedora-coreos-tracker#293.

The original plan for this functionality was to have it in config-bot:
coreos/fedora-coreos-releng-automation#48

But in the end, I think it's more natural to have it as a Jenkins job
given that it does a lot of the same things as the pipeline/upstream CI
jobs. So that way it looks and feels just like another job that runs
cosa, and we get kola artifacts, we can re-use the shared library,
it's easily inspectable, we can hook it to Slack, etc...
jlebon added a commit to jlebon/fedora-coreos-pipeline that referenced this pull request May 1, 2020
This job will implement lockfile bumping for testing-devel and
next-devel: coreos/fedora-coreos-tracker#293.

The original plan for this functionality was to have it in config-bot:
coreos/fedora-coreos-releng-automation#48

But in the end, I think it's more natural to have it as a Jenkins job
given that it does a lot of the same things as the pipeline/upstream CI
jobs. So that way it looks and feels just like another job that runs
cosa, and we get kola artifacts, we can re-use the shared library,
it's easily inspectable, we can hook it to Slack, etc...
jlebon added a commit to jlebon/fedora-coreos-pipeline that referenced this pull request May 1, 2020
This job will implement lockfile bumping for testing-devel and
next-devel: coreos/fedora-coreos-tracker#293.

The original plan for this functionality was to have it in config-bot:
coreos/fedora-coreos-releng-automation#48

But in the end, I think it's more natural to have it as a Jenkins job
given that it does a lot of the same things as the pipeline/upstream CI
jobs. So that way it looks and feels just like another job that runs
cosa, and we get kola artifacts, we can re-use the shared library,
it's easily inspectable, we can hook it to Slack, etc...
jlebon added a commit to jlebon/fedora-coreos-pipeline that referenced this pull request May 1, 2020
This job will implement lockfile bumping for testing-devel and
next-devel: coreos/fedora-coreos-tracker#293.

The original plan for this functionality was to have it in config-bot:
coreos/fedora-coreos-releng-automation#48

But in the end, I think it's more natural to have it as a Jenkins job
given that it does a lot of the same things as the pipeline/upstream CI
jobs. So that way it looks and feels just like another job that runs
cosa, and we get kola artifacts, we can re-use the shared library,
it's easily inspectable, we can hook it to Slack, etc...
jlebon added a commit to jlebon/fedora-coreos-pipeline that referenced this pull request May 4, 2020
This job will implement lockfile bumping for testing-devel and
next-devel: coreos/fedora-coreos-tracker#293.

The original plan for this functionality was to have it in config-bot:
coreos/fedora-coreos-releng-automation#48

But in the end, I think it's more natural to have it as a Jenkins job
given that it does a lot of the same things as the pipeline/upstream CI
jobs. So that way it looks and feels just like another job that runs
cosa, and we get kola artifacts, we can re-use the shared library,
it's easily inspectable, we can hook it to Slack, etc...
jlebon added a commit to jlebon/fedora-coreos-pipeline that referenced this pull request May 4, 2020
This job will implement lockfile bumping for testing-devel and
next-devel: coreos/fedora-coreos-tracker#293.

The original plan for this functionality was to have it in config-bot:
coreos/fedora-coreos-releng-automation#48

But in the end, I think it's more natural to have it as a Jenkins job
given that it does a lot of the same things as the pipeline/upstream CI
jobs. So that way it looks and feels just like another job that runs
cosa, and we get kola artifacts, we can re-use the shared library,
it's easily inspectable, we can hook it to Slack, etc...
jlebon added a commit to jlebon/fedora-coreos-pipeline that referenced this pull request May 4, 2020
This job will implement lockfile bumping for testing-devel and
next-devel: coreos/fedora-coreos-tracker#293.

The original plan for this functionality was to have it in config-bot:
coreos/fedora-coreos-releng-automation#48

But in the end, I think it's more natural to have it as a Jenkins job
given that it does a lot of the same things as the pipeline/upstream CI
jobs. So that way it looks and feels just like another job that runs
cosa, and we get kola artifacts, we can re-use the shared library,
it's easily inspectable, we can hook it to Slack, etc...
jlebon added a commit to jlebon/fedora-coreos-pipeline that referenced this pull request May 4, 2020
This job will implement lockfile bumping for testing-devel and
next-devel: coreos/fedora-coreos-tracker#293.

The original plan for this functionality was to have it in config-bot:
coreos/fedora-coreos-releng-automation#48

But in the end, I think it's more natural to have it as a Jenkins job
given that it does a lot of the same things as the pipeline/upstream CI
jobs. So that way it looks and feels just like another job that runs
cosa, and we get kola artifacts, we can re-use the shared library,
it's easily inspectable, we can hook it to Slack, etc...
jlebon added a commit to coreos/fedora-coreos-pipeline that referenced this pull request May 5, 2020
This job will implement lockfile bumping for testing-devel and
next-devel: coreos/fedora-coreos-tracker#293.

The original plan for this functionality was to have it in config-bot:
coreos/fedora-coreos-releng-automation#48

But in the end, I think it's more natural to have it as a Jenkins job
given that it does a lot of the same things as the pipeline/upstream CI
jobs. So that way it looks and feels just like another job that runs
cosa, and we get kola artifacts, we can re-use the shared library,
it's easily inspectable, we can hook it to Slack, etc...
@jlebon
Copy link
Member Author

jlebon commented May 5, 2020

Replaced by coreos/fedora-coreos-pipeline#229.

@jlebon jlebon closed this May 5, 2020
@jlebon jlebon deleted the pr/bump-lockfiles branch April 22, 2023 22:47
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