search for: notfounderror

...the trace. Defining a separate exception for "no bootloader config file found" avoids this. diff --git a/tools/pygrub/src/pygrub b/tools/pygrub/src/pygrub --- a/tools/pygrub/src/pygrub +++ b/tools/pygrub/src/pygrub @@ -29,6 +29,9 @@ import grub.ExtLinuxConf PYGRUB_VER = 0.6 +class NotFoundError(RuntimeError): + pass + def enable_cursor(ison): if ison: val = 2 @@ -412,7 +412,7 @@ class Grub: self.cf.filename = f break if self.__dict__.get(''cf'', None) is None: - raise RuntimeError, "couldn'...

...fer " + "to finalize") + continue + + raise RuntimeError("Unexpected transfer phase while finalizing " + "upload %r" % transfer.phase) except sdk.NotFoundError: pass --

...r " >> - "to finalize") >> + transfer = transfer_service.get() >> + >> + if transfer is None: > > > Are you sure this is possible? the original code assumed that we > fail with sdk.NotFoundError. > >> >> + disk_service = h['disk_service'] >> + disk = disk_service.get() >> + if disk.status == types.DiskStatus.OK: >> + continue > > > If the disk status is OK the u...

This is a patch which Daniel Erez wrote originally. I have modified it only to fix a small bug in the debug() statement, and Ilanit kindly tested it here: https://bugzilla.redhat.com/show_bug.cgi?id=1680361#c38 Rich.

..._disk_uuid') > + if uuid is not None: > + try: > + disks_service.disk_service(uuid).get() > + nbdkit.set_error(errno.EEXIST) > + raise ValueError("Disk with the UUID '%s' already exists" % uuid) > + except sdk.NotFoundError: > + pass This check seems correct to me, although it is done too late: IMHO this is something to do in the precheck script, so we do not even attempt to connect to the source if any of the specified UUIDs already exists in oVirt. -- Pino Toscano

...) if time.time() > endt: - raise RuntimeError("timed out waiting for transfer " + + raise RuntimeError("timed out waiting for transfer " "to finalize") except sdk.NotFoundError: pass -- 2.17.1

..."to finalize") > + continue > + if disk.status == types.DiskStatus.OK: > + debug("finalized after %s seconds", time.time() - > start) > + break > except sdk.NotFoundError: > + debug("transfer finalize failed for disk: %s", disk_id) > This should explain the failure: "transfer failed: disk %s not found" > pass > This should be a failure now. > > # Write the disk ID file. Only do this o...

...;timed out waiting for transfer " > - "to finalize") > + transfer = transfer_service.get() > + > + if transfer is None: > Are you sure this is possible? the original code assumed that we fail with sdk.NotFoundError. > + disk_service = h['disk_service'] > + disk = disk_service.get() > + if disk.status == types.DiskStatus.OK: > + continue > If the disk status is OK the upload was finished, so we should brea...

...ord" with "Lmmsoht" # features/ step_definitions/webrat_steps.rb:12 And I fill in "password_confirmation" with "Lmmsoht" # features/ step_definitions/webrat_steps.rb:12 Could not find field: "password_confirmation" (Webrat::NotFoundError) The test passes if I prepend "user_" to the field name ie. "user_password_confirmation". Same applies to the ''given_names'' field. Is this a bug or am I making an incorrect assumption on how things are supposed to work? TIA, JohnM -- "I'...

..."to finalize") > + continue > + if disk.status == types.DiskStatus.OK: > + debug("finalized after %s seconds" % (time.time() - > start)) > + break > except sdk.NotFoundError: > - pass > + raise RuntimeError("transfer failed: disk %s not found" % > disk_id) > > # Write the disk ID file. Only do this on successful completion. > with builtin_open(params['diskid_file'], 'w') as fp: > --...

...ed in successfully") end Then /^"([^"]*)" link should be shown$/ do |arg1| clicks_link user_list_path #pending # express the regexp above with the code you wish you had end My problem is 2nd step:- Could not find link with text or title or id "/user-list" (Webrat::NotFoundError) features/admin.feature:5:in `Then "Users" link should be shown'' Failing Scenarios: cucumber features/admin.feature:3 # Scenario: Login as Admin Is there anything wrong with my code? How can I check whether the link is present or not? Also, My login is successful as admin but...

My stab v2 at fixing this: https://bugzilla.redhat.com/show_bug.cgi?id=1789279 It took me quite some time to go through the whole rfc 4122 just to realize we do not need to do anything with the versions. v2: - Use EEXIST instead of EINVAL - Put the colliding UUID into the error - Do not evaluate the PCRE needlessly multiple times v1:

...sk and transfer shortly after. + endt = time.time() + timeout while True: - transfer = transfer_service.get() - if transfer.phase != types.ImageTransferPhase.INITIALIZING: + time.sleep(1) + try: + transfer = transfer_service.get() + except sdk.NotFoundError: + # The system has removed the disk and the transfer. + raise RuntimeError("transfer %s was removed" % transfer.id) + + if transfer.phase == types.ImageTransferPhase.FINISHED_FAILURE: + # The system will remove the disk and the transfer soon. +...

My stab at fixing this: https://bugzilla.redhat.com/show_bug.cgi?id=1789279 It took me quite some time to go through the whole rfc 4122 just to realize we do not need to do anything with the versions. Martin Kletzander (2): rhv-upload: Validate UUIDs passed to -oo rhv-disk-uuid (RHBZ#1789279) rhv-upload: Check that rhv-disk-uuid is not already taken (RHBZ#1789279)

...U General Public License along +# with this program; if not, write to the Free Software Foundation, Inc., +# 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA. + +# Fake ovirtsdk4 module used as a test harness. +# See v2v/test-v2v-o-rhv-upload.sh + +class Error(Exception): + pass +class NotFoundError(Error): + pass + +class Connection(object): + def __init__( + self, + url = None, + username = None, + password = None, + ca_file = None, + log = None, + insecure = False, + ): + pass + + def system_serv...

..."to finalize") > + continue > + if disk.status == types.DiskStatus.OK: > + debug("finalized after %s seconds", time.time() - start) > + break > except sdk.NotFoundError: > - pass > + raise RuntimeError("transfer failed: disk %s not found" % disk_id) > > # Write the disk ID file. Only do this on successful completion. > with builtins.open(params['diskid_file'], 'w') as fp: Ilanit te...

v1 was here: https://www.redhat.com/archives/libguestfs/2018-September/msg00121.html v2: - Rewrote patch 2 from scratch so it incorporates Nir's suggestions. - Add fake module to EXTRA_DIST. - Retested. Unfortunately I am no longer able to test the ordinary conversion path because ovirtsdk4 is incompatible with Fedora 29 / Python 3.7:

...f time.time() > endt: > - raise RuntimeError("timed out waiting for transfer " + > + raise RuntimeError("timed out waiting for transfer " > "to finalize") > except sdk.NotFoundError: > pass > -- > 2.17.1 -- Richard Jones, Virtualization Group, Red Hat http://people.redhat.com/~rjones Read my programming and virtualization blog: http://rwmj.wordpress.com virt-builder quickly builds VMs from scratch http://libguestfs.org/virt-builder.1.html

...er " + "to finalize") + continue + if disk.status == types.DiskStatus.OK: + debug("finalized after %s seconds", time.time() - start) + break except sdk.NotFoundError: + debug("transfer finalize failed for disk: %s", disk_id) pass # Write the disk ID file. Only do this on successful completion. --

...er " + "to finalize") + continue + if disk.status == types.DiskStatus.OK: + debug("finalized after %s seconds", time.time() - start) + break except sdk.NotFoundError: - pass + raise RuntimeError("transfer failed: disk %s not found" % disk_id) # Write the disk ID file. Only do this on successful completion. with builtins.open(params['diskid_file'], 'w') as fp: --