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

libdnf: Bump and disable html and man pages #1463

Closed
wants to merge 3 commits into from

Conversation

jlebon
Copy link
Member

@jlebon jlebon commented Jul 17, 2018

Skip building man pages and HTML docs for our embedded libdnf to speed
up builds.

See: projectatomic/libdnf#3

Update submodule: libdnf

@cgwalters
Copy link
Member

If we're bumping libdnf...what about merging projectatomic/libdnf#2 too?

@cgwalters
Copy link
Member

(It's OK if you tell me to just wait until we rebase libdnf...that patch has risk but also definite reward for Fedora today)

Skip building man pages and HTML docs for our embedded libdnf to speed
up builds.

This bump also pulls in a fix to ensure we never try to install src
packages from `dnf_context_install()`.

See: projectatomic/libdnf#3

Update submodule: libdnf
@jlebon jlebon force-pushed the pr/libdnf-minor-bump branch from a7cc4ce to 6270f0e Compare July 17, 2018 18:10
@jlebon
Copy link
Member Author

jlebon commented Jul 17, 2018

I was hesitating a bit because of the additional changes that were made upstream before it was merged. I wanted to make we weren't going to subtly change behaviour again once we do rebase (not that we won't have plenty of other behaviour changes I'm sure). Though looking at it carefully, it seems like that shouldn't be the case.

@cgwalters
Copy link
Member

@rh-atomic-bot r+ 6270f0e

@rh-atomic-bot
Copy link

⌛ Testing commit 6270f0e with merge f91d2a1...

rh-atomic-bot pushed a commit that referenced this pull request Jul 23, 2018
Skip building man pages and HTML docs for our embedded libdnf to speed
up builds.

This bump also pulls in a fix to ensure we never try to install src
packages from `dnf_context_install()`.

See: projectatomic/libdnf#3

Update submodule: libdnf

Closes: #1463
Approved by: cgwalters
@cgwalters
Copy link
Member

@rh-atomic-bot r+ 4fc34ca

@rh-atomic-bot
Copy link

⌛ Testing commit 4fc34ca with merge 5d2e52c...

rh-atomic-bot pushed a commit that referenced this pull request Jul 23, 2018
Skip building man pages and HTML docs for our embedded libdnf to speed
up builds.

This bump also pulls in a fix to ensure we never try to install src
packages from `dnf_context_install()`.

See: projectatomic/libdnf#3

Update submodule: libdnf

Closes: #1463
Approved by: cgwalters
@rh-atomic-bot
Copy link

💔 Test failed - status-atomicjenkins

@jlebon
Copy link
Member Author

jlebon commented Jul 23, 2018

@rh-atomic-bot retry

@rh-atomic-bot
Copy link

⌛ Testing commit 4fc34ca with merge 9a62b86...

rh-atomic-bot pushed a commit that referenced this pull request Jul 23, 2018
Skip building man pages and HTML docs for our embedded libdnf to speed
up builds.

This bump also pulls in a fix to ensure we never try to install src
packages from `dnf_context_install()`.

See: projectatomic/libdnf#3

Update submodule: libdnf

Closes: #1463
Approved by: cgwalters
@jlebon
Copy link
Member Author

jlebon commented Jul 23, 2018

@rh-atomic-bot retry

rh-atomic-bot pushed a commit that referenced this pull request Jul 23, 2018
Skip building man pages and HTML docs for our embedded libdnf to speed
up builds.

This bump also pulls in a fix to ensure we never try to install src
packages from `dnf_context_install()`.

See: projectatomic/libdnf#3

Update submodule: libdnf

Closes: #1463
Approved by: cgwalters
@rh-atomic-bot
Copy link

⌛ Testing commit 4fc34ca with merge c2da386...

@jlebon
Copy link
Member Author

jlebon commented Jul 24, 2018

@rh-atomic-bot retry

@rh-atomic-bot
Copy link

⌛ Testing commit 4fc34ca with merge 27ccfcb...

rh-atomic-bot pushed a commit that referenced this pull request Jul 24, 2018
Skip building man pages and HTML docs for our embedded libdnf to speed
up builds.

This bump also pulls in a fix to ensure we never try to install src
packages from `dnf_context_install()`.

See: projectatomic/libdnf#3

Update submodule: libdnf

Closes: #1463
Approved by: cgwalters
@rh-atomic-bot
Copy link

💔 Test failed - status-atomicjenkins

We're consistently seeing timeouts on the `f28-compose` test. It's
likely due to a performance regression in the backend rather than
anything else. I filed a ticket to get an investigation going, though
for now at least, let's bump the timeout some more to keep patches
flowing.
@jlebon
Copy link
Member Author

jlebon commented Jul 24, 2018

OK, I'm fairly sure at this point that we're experiencing some general I/O degradation. I created a ticket in the relevant tracker, though for the time being I've increased the timeout to get PRs merged.

@rh-atomic-bot r+ 50e9ade

@rh-atomic-bot
Copy link

⌛ Testing commit 50e9ade with merge a3612f2...

rh-atomic-bot pushed a commit that referenced this pull request Jul 24, 2018
Skip building man pages and HTML docs for our embedded libdnf to speed
up builds.

This bump also pulls in a fix to ensure we never try to install src
packages from `dnf_context_install()`.

See: projectatomic/libdnf#3

Update submodule: libdnf

Closes: #1463
Approved by: jlebon
rh-atomic-bot pushed a commit that referenced this pull request Jul 24, 2018
We're consistently seeing timeouts on the `f28-compose` test. It's
likely due to a performance regression in the backend rather than
anything else. I filed a ticket to get an investigation going, though
for now at least, let's bump the timeout some more to keep patches
flowing.

Closes: #1463
Approved by: jlebon
@rh-atomic-bot
Copy link

💔 Test failed - status-atomicjenkins

@cgwalters
Copy link
Member

Ugh man, this time GNOME gitlab 500'ing. That said it's hard to escape the fact that what we really should be doing is maintaining local cache/mirrors of these - gets into the rdgo vs papr style discussions.

@jlebon
Copy link
Member Author

jlebon commented Jul 24, 2018

@rh-atomic-bot retry

rh-atomic-bot pushed a commit that referenced this pull request Jul 24, 2018
Skip building man pages and HTML docs for our embedded libdnf to speed
up builds.

This bump also pulls in a fix to ensure we never try to install src
packages from `dnf_context_install()`.

See: projectatomic/libdnf#3

Update submodule: libdnf

Closes: #1463
Approved by: jlebon
rh-atomic-bot pushed a commit that referenced this pull request Jul 24, 2018
We're consistently seeing timeouts on the `f28-compose` test. It's
likely due to a performance regression in the backend rather than
anything else. I filed a ticket to get an investigation going, though
for now at least, let's bump the timeout some more to keep patches
flowing.

Closes: #1463
Approved by: jlebon
@rh-atomic-bot
Copy link

⌛ Testing commit 50e9ade with merge cacdb2a...

@jlebon
Copy link
Member Author

jlebon commented Jul 24, 2018

Looks like gitlab.gnome.org is up now. No wait, it's down again... Ahh, it's just slow to load.

@rh-atomic-bot
Copy link

💔 Test failed - status-atomicjenkins

@jlebon
Copy link
Member Author

jlebon commented Jul 24, 2018

@rh-atomic-bot retry

@rh-atomic-bot
Copy link

⌛ Testing commit 50e9ade with merge 1a82f21...

rh-atomic-bot pushed a commit that referenced this pull request Jul 24, 2018
We're consistently seeing timeouts on the `f28-compose` test. It's
likely due to a performance regression in the backend rather than
anything else. I filed a ticket to get an investigation going, though
for now at least, let's bump the timeout some more to keep patches
flowing.

Closes: #1463
Approved by: jlebon
@rh-atomic-bot
Copy link

☀️ Test successful - status-atomicjenkins
Approved by: jlebon
Pushing 1a82f21 to master...

@cgwalters
Copy link
Member

One upside to periodic CI outages is that I am unreasonably happy and excited when the commits are flowing again!

@jlebon jlebon deleted the pr/libdnf-minor-bump branch April 23, 2023 23:32
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.

3 participants