Skip to content

Fix trigger of functional-dns job#2364

Merged
EmilienM merged 1 commit intogophercloud:masterfrom
shiftstack:fix-trigger-dns-job
Feb 28, 2022
Merged

Fix trigger of functional-dns job#2364
EmilienM merged 1 commit intogophercloud:masterfrom
shiftstack:fix-trigger-dns-job

Conversation

@mandre
Copy link
Copy Markdown
Contributor

@mandre mandre commented Feb 26, 2022

The previous filter would not run the functional-dns job when files
touching the DNS service, under the openstack/dns path, were modified.
This commit fixes it.

We can't use the simpler **dns** filter as this would match the files
for the neutron DNS extension which is a different thing.

The previous filter would not run the functional-dns job when files
touching the DNS service, under the openstack/dns path, were modified.
This commit fixes it.

We can't use the simpler `**dns**` filter as this would match the files
for the neutron DNS extension which is a different thing.
@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage remained the same at 79.893% when pulling 9971790 on shiftstack:fix-trigger-dns-job into 8b22827 on gophercloud:master.

@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci bot commented Feb 26, 2022

Build failed.

@mandre
Copy link
Copy Markdown
Contributor Author

mandre commented Feb 28, 2022

Build failed.
[...]
* gophercloud-acceptance-test-networking : FAILURE in 58m 30s

Why the heck is gophercloud-acceptance-test-networking running? I thought we removed it with #2351.

@EmilienM EmilienM merged commit b14a81c into gophercloud:master Feb 28, 2022
@EmilienM EmilienM deleted the fix-trigger-dns-job branch February 28, 2022 18:36
@EmilienM
Copy link
Copy Markdown
Contributor

Build failed.
[...]

Why the heck is gophercloud-acceptance-test-networking running? I thought we removed it with #2351.

no idea

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.

3 participants