Don't stuff non-db data into instance dict
Don't stuff non-db data into instance dict

The RPC API passes admin_pass and injected_files as arguments, but when
calling the driver API, those values get stuffed into the instance dict
which is otherwise all data that is stored in the database. This has
led to some bugs (such as 1034948) which could have been avoided if
the code was consistent.

Change-Id: I09a7de8eca3791a5ab27e93b22e5ccccee31cf09

ionutbalutoiu/ironic,ionutbalutoiu/ironic,NaohiroTamura/ironic,debayanray/ironic_backup,naototty/vagrant-lxc-ironic,ramineni/myironic,SauloAislan/ironic,citrix-openstack-build/ironic,faizan-barmawer/elytics,faizan-barmawer/openstack_ironic,dims/ironic,bacaldwell/ironic,emonty/ironic,supermari0/ironic,SauloAislan/ironic,openstack/ironic,JioCloud/ironic,openstack/ironic-lib,redhat-openstack/ironic,citrix-openstack-build/ironic-lib,pshchelo/ironic,naterh/ironic,citrix-openstack-build/ironic,rdo-management/ironic,bacaldwell/ironic,dims/ironic,openstack/ironic,devananda/ironic,NaohiroTamura/ironic-lib,NaohiroTamura/ironic,Tehsmash/ironic,faizan-barmawer/ironic-lib,Tan0/ironic,rackerlabs/ironic,varunarya10/ironic,naototty/vagrant-lxc-ironic,hpproliant/ironic,pshchelo/ironic,froyobin/ironic